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

Change case sensitivity #22

Closed
wants to merge 0 commits into from
Closed

Change case sensitivity #22

wants to merge 0 commits into from

Conversation

JeodC
Copy link
Collaborator

@JeodC JeodC commented Apr 16, 2024

Converts all directories and files to lowercase except for top level files. May have conflicts with some recent commits--resolve manually. This was done via python script:

import os

def convert_to_lowercase(directory):
    for root, dirs, files in os.walk(directory):
        for file in files:
            original_path = os.path.join(root, file)
            lowercase_path = original_path.lower()
            if original_path != lowercase_path:
                os.rename(original_path, lowercase_path)
                print(f"Renamed '{original_path}' to '{lowercase_path}'")
        for dir in dirs:
            original_path = os.path.join(root, dir)
            lowercase_path = original_path.lower()
            if original_path != lowercase_path:
                os.rename(original_path, lowercase_path)
                print(f"Renamed '{original_path}' to '{lowercase_path}'")

if __name__ == "__main__":
    script_dir = os.path.dirname(os.path.abspath(__file__))
    convert_to_lowercase(script_dir)

Ignore this PR if you want, it's a preference thing. Looking through the scripts directory was particularly annoying as there are Levelxxx.cpp and levelxxx.cpp files. ;)

@JeodC
Copy link
Collaborator Author

JeodC commented Apr 16, 2024

Looks like #19 was committed after I forked, so would need re-committing ig.

@JeodC
Copy link
Collaborator Author

JeodC commented Apr 16, 2024

I have added the missing commit to .clang-format and removed what appeared to be a tmp file #.clang-format#.

@winterheart
Copy link
Collaborator

CMakeLists.txt also renamed which shouldn't be. I think main idea is rename project files, if so, there should be file mask on *.cpp, *.CPP, *.c, *.C, *.h, *.H.

@JeodC
Copy link
Collaborator Author

JeodC commented Apr 16, 2024

CMakeLists.txt also renamed which shouldn't be. I think main idea is rename project files, if so, there should be file mask on *.cpp, *.CPP, *.c, *.C, *.h, *.H.

Yes, this was quick n' dirty. If you'd like to do this along with your clang formatting, go ahead.

@JeodC
Copy link
Collaborator Author

JeodC commented Apr 16, 2024

This is now fixed.

@JeodC JeodC closed this Apr 17, 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