Skip to content

Conversation

@hanjie-chen
Copy link
Contributor

Description

I added some explanation about the ignore attribute in the watch rule. I noticed the document says:

All paths are relative to the project directory

However, when I deployed it in my project, I found that my container was rebuilding crazily based on the action I defined.

After testing, I discovered that the ignore path is actually relative to the path defined in the same watch rule, not the project directory. I also confirmed this with the AI on the Docker Docs website.

This PR updates the documentation to clarify this behavior.

add ignore path which is the relative to the path defined in teh same watch
@github-actions github-actions bot added the area/compose Relates to docker-compose.yml spec or docker-compose binary label Jan 15, 2025
@netlify
Copy link

netlify bot commented Jan 15, 2025

Deploy Preview for docsdocker ready!

Name Link
🔨 Latest commit ae86626
🔍 Latest deploy log https://app.netlify.com/sites/docsdocker/deploys/67a081ade09297000833c39a
😎 Deploy Preview https://deploy-preview-21820--docsdocker.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@hanjie-chen
Copy link
Contributor Author

hi, @aevesdocker can you help review this change

@aevesdocker aevesdocker requested a review from glours January 20, 2025 10:15
@hanjie-chen
Copy link
Contributor Author

hi @glours can you help review this

Copy link
Contributor

@glours glours left a comment

Choose a reason for hiding this comment

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

@hanjie-chen @aevesdocker LGTM when the 2 comments I made will be addressed

Co-authored-by: Guillaume Lours <705411+glours@users.noreply.github.com>
Copy link
Contributor

@aevesdocker aevesdocker left a comment

Choose a reason for hiding this comment

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

Thanks @hanjie-chen, and thanks @glours for the review

@aevesdocker aevesdocker merged commit 01a0f97 into docker:main Feb 3, 2025
14 checks passed
@hanjie-chen
Copy link
Contributor Author

Thanks @aevesdocker and @glours for the reviews !

@hanjie-chen hanjie-chen deleted the patch-1 branch February 5, 2025 02:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/compose Relates to docker-compose.yml spec or docker-compose binary

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants