Skip to content

Conversation

@GordonSmith
Copy link
Member

@GordonSmith GordonSmith commented Nov 15, 2025

Checklist:

  • The commit message is properly formatted and free of typos.
    • The commit message title makes sense in a changelog, by itself.
    • The commit message includes a "fixes" reference if appropriate.
    • The commit is signed.
  • The change has been fully tested:
    • I have viewed all related gallery items
    • I have viewed all related dermatology items
  • There are no similar instances of the same problem that should be addressed
    • I have addressed them here
    • I have raised new issues to address them separately

Testing:

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the dataflow package documentation by correcting test file paths in the README and adding comprehensive Copilot instructions for package contributors.

  • Updates all test reference links in README from old src/__tests__/*.ts paths to new tests/*.spec.ts paths
  • Adds a new .github/copilot-instructions.md file documenting architecture, patterns, and development workflows

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
packages/dataflow/README.md Updates 28 test file reference links to point to the correct tests/*.spec.ts location after test directory restructuring
packages/dataflow/.github/copilot-instructions.md Adds comprehensive documentation covering architecture, coding patterns, build workflows, and TypeScript configuration

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@GordonSmith GordonSmith force-pushed the DOCS branch 2 times, most recently from c1d3d41 to 636e38e Compare November 26, 2025 14:26
Signed-off-by: Gordon Smith <GordonJSmith@gmail.com>
@GordonSmith GordonSmith merged commit 1c3b1e2 into hpcc-systems:main Nov 26, 2025
3 checks passed
@GordonSmith GordonSmith deleted the DOCS branch November 26, 2025 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant