Skip to content

fix(cli): respect .gitignore patterns in dev watcher#325

Closed
noahkhomer18 wants to merge 1 commit into
withastro:mainfrom
noahkhomer18:fix/316-watcher-ignores
Closed

fix(cli): respect .gitignore patterns in dev watcher#325
noahkhomer18 wants to merge 1 commit into
withastro:mainfrom
noahkhomer18:fix/316-watcher-ignores

Conversation

@noahkhomer18

Copy link
Copy Markdown

Generated project directories like evals/results/ or custom tool output folders are ignored by .gitignore but still trigger rebuild churn in flue dev because the watcher doesn't consult it. Now reads .gitignore at watcher creation and skips paths matching any of its patterns.

Closes #316

Generated project directories like evals/results/ or custom tool
output folders are ignored by .gitignore but still trigger rebuild
churn in flue dev because the watcher doesn't consult it. Now reads
.gitignore at watcher creation and skips paths matching any of its
patterns.

Closes withastro#316
@astrobot-houston

Copy link
Copy Markdown

Thanks for the contribution! We're closing this PR and moving the conversation to the issue: #326

We've moved to a model where bugs and feature proposals are discussed in issues/discussions before code review, so the community can help prioritize and shape the work. Your branch is linked from the new thread so the implementation isn't lost — please join us there to continue the conversation.

— astrobot 🤖

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.

flue dev needs configurable watcher ignores for generated project directories

2 participants