-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Watch: Support Path Pattern Matching #12544
Comments
I would propose to introduce a new keyword Or we can change |
@jhrotko I guess it's a tradeoff between making the API as smooth as possible for the user and an implementation that actually achieves the desired behaviour. I am guess that extending the behaviour of |
I created a POC in these branches (feature-wild-star-watch-path): in these branches the implementation is under the If you want to try, clone my fork/branch and run make build. in your
you can run with the specific compose build (from the make build) by running
|
On #12584 I proposed to keep using a single develop:
watch:
- action: rebuild
path: frontend
include:
- *.json
- *.js |
Request
It would be nice if the watch
path
attribute behaved like theignore
attribute. For example,path
could accept wildcards such as*
.At the very least it would help to have some more detail in the API documentation adding that (at least for the time being) pattern matching does not work on
path
.Reference: https://docs.docker.com/reference/compose-file/develop/
Example
I would like the convenience of being able to do this:
instead of the verbose (yet explicit):
Nice To Have
Bonus points for allowing multiple
path
entries in a singlerebuild
action:Currently, passing a space separated list is silently interpreted as a single path. So, the below example (misleadingly) does not match
frontend/package.json
norfrontend/vite.config.js
The text was updated successfully, but these errors were encountered: