-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feat: ignore vim *.swp files in hot-reload example #6895
Conversation
It is not obvious that `skaffold dev` will rebuild the image even when a file that is not mapped in `skaffold.yaml`'s manual sync config is added or changed. `skaffold dev` instead respects `.dockerignore` as I discovered [in this issue](GoogleContainerTools#3124). Add `**/*.swp` to both node and python directories in the hot-reload example so that this example is more likely to work out-of-the-box for new users. Signed-off-by: David Xia <dxia@spotify.com>
Codecov Report
@@ Coverage Diff @@
## main #6895 +/- ##
==========================================
- Coverage 70.48% 69.13% -1.35%
==========================================
Files 515 547 +32
Lines 23150 25081 +1931
==========================================
+ Hits 16317 17340 +1023
- Misses 5776 6576 +800
- Partials 1057 1165 +108
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
looks like kokoro is stuck preventing the auto-merge? |
hey @davidxia , sorry for the delay on this! I've triggered the kokoro build. I'll merge this when done assuming it goes green |
@MarlonGamez thank you! |
It is not obvious that
skaffold dev
will rebuild the imageeven when a file that is not mapped in
skaffold.yaml
'smanual sync config is added or changed.
skaffold dev
insteadrespects
.dockerignore
as I discoveredin this issue.
Add
**/*.swp
to both node and python directories in the hot-reloadexample so that this example is more likely to work out-of-the-box
for new users.