-
Notifications
You must be signed in to change notification settings - Fork 0
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
Fix: replace snake case file names with camelCase #316
Fix: replace snake case file names with camelCase #316
Conversation
Hi @undernagruzez can you help me understand why these checks failed? |
I suppose the error is related to the origin of the branch, but will check once again later |
Code looks good to me, I merged some code on the playground and this would need to be updated. But this can generally be merged once tested on the test server. |
And how can we do that? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'd need to resolve the conflicts, I don't think I can resolve them in the fork.
I don't really now 🤔 you renamed a lot of files by changing from |
I'm sorry, I have picked up a new project. I may try this some time later. |
Motivation and Context
Fixes #68 replacing snake_case with camelCase, for consistency.
Description
Renamed files and imports that were using snake_case
Steps for Testing
Switch to replaceSnakeCaseWithCamel branch and try npm run dev
Screenshots
No change in UI