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

Fixing bug in filenames with dashes #4

Merged
merged 2 commits into from Dec 15, 2016
Merged

Conversation

msmith491
Copy link
Contributor

@msmith491 msmith491 commented Dec 12, 2016

Filenames with dashes would not create valid rust identifiers since they
cannot include dashes in the identifier.

Additionally added attribute to turn off unused code warnings and moved
the #[allow(non_upper_case_globals)] to a global attribute

msmith491 and others added 2 commits December 12, 2016 13:10
Filenames with dasesh would not create valid rust identifiers since they
cannot include dashes in the identifier.

Additionally added attribute to turn off unused code warnings and moved
the #[allow(non_upper_case_globals)] to a global attribute
@pyrossh
Copy link
Owner

pyrossh commented Dec 15, 2016

Thanks for the PR. Yeah this solves the specific case for dash maybe there is a lib like lodash for rust which will remove all special characters like snakeCaseify or camelCaseify.

@pyrossh pyrossh merged commit 7169ff1 into pyrossh:master Dec 15, 2016
@pyrossh
Copy link
Owner

pyrossh commented Dec 15, 2016

LGTM

pyrossh pushed a commit that referenced this pull request Aug 24, 2021
Fixing bug in filenames with dashes
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.

None yet

2 participants