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

Decide on proper casing of AutoGenBench folder names, and make changes as needed. #1424

Closed
Tracked by #973
afourney opened this issue Jan 27, 2024 · 3 comments
Closed
Tracked by #973
Assignees
Labels
proj-autogenbench Issues related to AutoGenBench.

Comments

@afourney
Copy link
Member

afourney commented Jan 27, 2024

See: #1048 (comment)

@jackgerrits
Copy link
Member

I would vote for all lowercase especially since not all OS filesystems support case sensitivity

@afourney
Copy link
Member Author

I would vote for all lowercase especially since not all OS filesystems support case sensitivity

But OS filesystem case sensitivity only matters if there are two or more files of folders with names that differ only by case. We will have upper case or mixed case names like README.md and Dockerfile already. I think there is a case to be made about standardization, or about what is "pythonic", but I don't think there's any risk of breaking anything.

@jackgerrits
Copy link
Member

good point, totally agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
proj-autogenbench Issues related to AutoGenBench.
Projects
None yet
Development

No branches or pull requests

4 participants