Skip to content
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

Texture replacement: Prioritize ini file [hashes] section over just files in the "root" folder. #18466

Merged
merged 1 commit into from
Dec 3, 2023

Conversation

hrydgard
Copy link
Owner

@hrydgard hrydgard commented Dec 2, 2023

This reverts back to what the old behavior seems to have been, as detailed in #18465 .

I think this does make more sense, too.

This reverts back to the old behavior, as mentioned in #18465
@hrydgard hrydgard added this to the v1.17.0 milestone Dec 2, 2023
Copy link
Collaborator

@unknownbrackets unknownbrackets left a comment

Choose a reason for hiding this comment

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

Yeah, that behavior was previously very intentional. I mean, it feels obvious - you're specifying the path for the files in the ini, so the default behavior shouldn't even activate.

-[Unknown]

@hrydgard
Copy link
Owner Author

hrydgard commented Dec 3, 2023

Yeah, my bad for screwing this up, it seems.

@hrydgard hrydgard merged commit 8c0b0ff into master Dec 3, 2023
18 checks passed
@hrydgard hrydgard deleted the reverse-replacement-priorities branch December 3, 2023 00:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants