Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

filewatch: apply global ignores to config file watches #4310

Merged
merged 1 commit into from
Mar 12, 2021

Conversation

milas
Copy link
Contributor

@milas milas commented Mar 12, 2021

This was a regression in v0.18.12 - the global ignores were no
longer getting applied to the configs:singleton target for the
config files and/or anything explicitly watched with watch_file.

Closes #4308.

@milas milas added the bug Something isn't working label Mar 12, 2021
@milas milas requested review from nicks and landism March 12, 2021 18:13
This was a regression in v0.18.12 - the global ignores were no
longer getting applied to the `configs:singleton` target for the
config files and/or anything explicitly watched with `watch_file`.

Closes #4308.
@milas milas force-pushed the milas/bugfix-watch-ignores branch from 91139d6 to e1add8f Compare March 12, 2021 18:15
Copy link
Member

@nicks nicks left a comment

Choose a reason for hiding this comment

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

LGTM!

@milas milas merged commit 969b62d into master Mar 12, 2021
@milas milas deleted the milas/bugfix-watch-ignores branch March 12, 2021 18:49
milas added a commit that referenced this pull request Mar 12, 2021
This was a regression in v0.18.12 - the global ignores were no
longer getting applied to the `configs:singleton` target for the
config files and/or anything explicitly watched with `watch_file`.

This is a cherry-pick of 969b62d.
@milas
Copy link
Contributor Author

milas commented Mar 12, 2021

Merge to master: 969b62d
Cherry-pick for v0.18.13: 029dee5

pull bot pushed a commit to rizalgowandy/tilt that referenced this pull request Mar 12, 2021
This was a regression in v0.18.12 - the global ignores were no
longer getting applied to the `configs:singleton` target for the
config files and/or anything explicitly watched with `watch_file`.

Closes tilt-dev#4308.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

.tiltignore not applied to config files
2 participants