Skip to content

Ignore Claude worktrees in Flow and Jest - #1883

Closed
robhogan wants to merge 1 commit into
mainfrom
pr1883
Closed

Ignore Claude worktrees in Flow and Jest#1883
robhogan wants to merge 1 commit into
mainfrom
pr1883

Conversation

@robhogan

@robhogan robhogan commented Aug 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Ignore .claude directories during Flow checks.
  • Ignore .claude directories during Jest module discovery.
  • Prevent nested Claude worktrees from causing duplicate module and test discovery errors.

Test plan

  • yarn flow
  • yarn jest --runInBand

Pass locally even with a nested worktree containing another copy of Metro. They also work inside the worktree (under .claude), because exclusions are anchored after the project root.

Changelog: Internal

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 26, 2026
## Summary

- Ignore `.claude` directories during Flow checks.
- Ignore `.claude` directories during Jest module discovery.
- Prevent nested Claude worktrees from causing duplicate module and test discovery errors.

## Test plan

- `yarn flow`
- `yarn jest --runInBand`

Both pass locally.

changelog: internal
@facebook-github-tools facebook-github-tools Bot added the Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. label Aug 26, 2026
@cortinico

Copy link
Copy Markdown
Contributor

@robhogan is the red CI expected?

@robhogan

robhogan commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator Author

No but it's unrelated - looks like we have an issue with the just-released latest Node.js, looking into it...

Oh it's a setup-node GHA bug, latest is resolving to an -alpha prerelease, which it shouldn't :/

@robhogan

Copy link
Copy Markdown
Collaborator Author

@cortinico CI was an upstream transient issue, re-running worked.

@meta-codesync

meta-codesync Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

@cortinico has imported this pull request. If you are a Meta employee, you can view this in D117675204.

@meta-codesync meta-codesync Bot closed this in a81ef86 Aug 27, 2026
@meta-codesync meta-codesync Bot added the Merged label Aug 27, 2026
@meta-codesync

meta-codesync Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

@cortinico merged this pull request in a81ef86.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants