Skip to content

fix(reset): Add cycle detection fix#706

Merged
glours merged 1 commit intocompose-spec:mainfrom
idsulik:issue-12235-3
Nov 1, 2024
Merged

fix(reset): Add cycle detection fix#706
glours merged 1 commit intocompose-spec:mainfrom
idsulik:issue-12235-3

Conversation

@idsulik
Copy link
Copy Markdown
Contributor

@idsulik idsulik commented Oct 31, 2024

Here docker/compose#12247 (comment) the user found one more case when it fires while it shouldn't.

  1. Decided to move the detector inside the alias condition, because that's the only place where the cycle can happen.
  2. Added one more test

Signed-off-by: Suleiman Dibirov <idsulik@gmail.com>
@idsulik
Copy link
Copy Markdown
Contributor Author

idsulik commented Nov 1, 2024

@glours hi! Check please, I think it's pretty important

@glours glours merged commit b9d3b1d into compose-spec:main Nov 1, 2024
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.

2 participants