Skip to content

fix: ignore Anchor generated directory#75

Open
MolhamHamwi wants to merge 1 commit into
Ghbounty:mainfrom
MolhamHamwi:fix/ignore-anchor-generated-dir
Open

fix: ignore Anchor generated directory#75
MolhamHamwi wants to merge 1 commit into
Ghbounty:mainfrom
MolhamHamwi:fix/ignore-anchor-generated-dir

Conversation

@MolhamHamwi
Copy link
Copy Markdown

Summary

  • add .anchor/ to generated directory prefixes in @ghbounty/diff-filter
  • cover both nested and top-level .anchor/ generated paths in classify tests

Verification

  • pnpm --filter @ghbounty/diff-filter test
  • pnpm --filter @ghbounty/diff-filter typecheck
  • pre-commit: pnpm typecheck
  • pre-commit: pnpm test

Closes #67

Copilot AI review requested due to automatic review settings May 11, 2026 05:48
@MolhamHamwi MolhamHamwi requested a review from tomazzi14 as a code owner May 11, 2026 05:48
@vercel
Copy link
Copy Markdown

vercel Bot commented May 11, 2026

@MolhamHamwi is attempting to deploy a commit to the weareghbounty-6269's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown

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

Updates @ghbounty/diff-filter to treat Anchor’s .anchor/ output directory as generated content so it’s ignored during diff filtering, reducing noise from Solana/Anchor build artifacts (per #67).

Changes:

  • Add .anchor/ to GENERATED_DIR_PREFIXES so both top-level and nested .anchor/ paths are filtered.
  • Add test cases to ensure .anchor/ paths are classified as generated_dir.

Reviewed changes

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

File Description
packages/diff-filter/src/patterns.ts Adds .anchor/ to the generated directory prefix list so Anchor artifacts are ignored.
packages/diff-filter/tests/classify.test.ts Adds coverage for both nested and top-level .anchor/ paths in generated-dir classification tests.

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

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.

diff-filter: ignore Anchor's .anchor/ generated directory

2 participants