Skip to content

Makes order of directory resolving deterministic across different OS#2099

Merged
jensens merged 3 commits intocookiecutter:mainfrom
RaulWCosta:turn-directory-resolve-order-deterministic
Nov 8, 2025
Merged

Makes order of directory resolving deterministic across different OS#2099
jensens merged 3 commits intocookiecutter:mainfrom
RaulWCosta:turn-directory-resolve-order-deterministic

Conversation

@RaulWCosta
Copy link
Copy Markdown

The order of items in the dirnames and filenames lists of os.walk appears to differ depending on which operating system is being used.

This PR aims at accounting for this behaviour so that cookiecutter's conflicts handling logic stays consistent across different operating systems.

@RaulWCosta
Copy link
Copy Markdown
Author

RaulWCosta commented Sep 17, 2024

Thanks for the approval @kurtmckee :)
it's a bit odd that Github still didn't unblock merging

@kurtmckee
Copy link
Copy Markdown
Member

I'm not an approved reviewer, which GitHub indicates by the gray checkbox:

image

On some other projects I contribute to, I get that sweet, sweet green checkbox:

image

Anybody can review any PRs anywhere, so I do. 👍

@RaulWCosta
Copy link
Copy Markdown
Author

Oh my bad, I didn't see the grey check. That's what I get for doing multiple things at once 😅

@jensens jensens merged commit e86f445 into cookiecutter:main Nov 8, 2025
7 checks passed
armandmasseaugit pushed a commit to armandmasseaugit/cookiecutter that referenced this pull request Nov 19, 2025
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.

3 participants