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

Convert Codebase to CamelCase #10847

Open
51 tasks
jNullj opened this issue Jan 24, 2025 · 2 comments
Open
51 tasks

Convert Codebase to CamelCase #10847

jNullj opened this issue Jan 24, 2025 · 2 comments
Labels
core Server, BaseService, GitHub auth, Shared helpers

Comments

@jNullj
Copy link
Member

jNullj commented Jan 24, 2025

Refactor the existing codebase to follow camelCase naming conventions.

  • Add backwards compatibility for snake_case params

List of services and file to change:

@jNullj jNullj added the core Server, BaseService, GitHub auth, Shared helpers label Jan 24, 2025
@jNullj
Copy link
Member Author

jNullj commented Jan 25, 2025

I think a good starting point would be to use this regex
grep -rlPzo 'handle\([^\)]+[a-z]+(_[a-z])+[:,]?[^\)]+\)' --include="*.service.js"
It finds all camel_case in our handle arguments

also added to the list from the comment above.
the regex does not find params defined in a const in another location (as in not at the handle definition)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Server, BaseService, GitHub auth, Shared helpers
Projects
None yet
Development

No branches or pull requests

2 participants