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

Add a case_conflict.py to check for case-only file conflicts. #574

Merged
merged 3 commits into from
Dec 19, 2022

Conversation

andynu
Copy link
Contributor

@andynu andynu commented Dec 18, 2022

This is confusing and can cause issues for folks running case insensitive filesystems (happens in windows sometimes).

If there is a conflict, the pages in question are output. And you get an non-zero exit status. E.g.

Conflict!
        ../../README.md
        ../../Readme.md

Additionally adds this check to the update_hub and check_content github workflows.

Related to #308

This is confusing and can cause issues for folks running case
insensitive filesystems (happens in windows sometimes).

If there is a conflict the pages in question are output. And you get an
non-zero exit status. E.g.

    Conflict!
            ../../README.md
            ../../Readme.md
…ript.

This will prevent the addition of new files where the only difference between it and an existing file is the case of the filename.
This should ensure human edits to files do not result in filenames that
differ only in the case of the filename.
@claremacrae claremacrae merged commit 9d06745 into obsidian-community:main Dec 19, 2022
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