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

🎨(api) convert camelCase variables to snake_case #436

Closed
wants to merge 2 commits into from

Conversation

lebaudantoine
Copy link
Contributor

Purpose

Ensures PEP8 compliance without disabling PyLint.

Proposal

Implemented camelCase aliases on attributes or routes' parameters.
Let me know if I am being too pedantic, and this breaking change is not worth it.

@lebaudantoine lebaudantoine changed the title 🎨(api) convert camelCase variable to snake_case 🎨(api) convert camelCase variables to snake_case Sep 14, 2023
@lebaudantoine lebaudantoine self-assigned this Sep 14, 2023
@lebaudantoine lebaudantoine force-pushed the fix/camelcase-naming branch 2 times, most recently from df40712 to 533e362 Compare September 18, 2023 08:29
Copy link
Contributor

@wilbrdt wilbrdt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 🎉

Copy link
Contributor

@quitterie-lcs quitterie-lcs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

src/ralph/api/routers/statements.py Outdated Show resolved Hide resolved
src/ralph/backends/http/async_lrs.py Show resolved Hide resolved
@quitterie-lcs
Copy link
Contributor

quitterie-lcs commented Sep 18, 2023

@lebaudantoine some camelCase remain in Pydantic models in models. To align with this PR, we could standardise all our Pydantic models, but that could be done in another PR

@lebaudantoine
Copy link
Contributor Author

Issue opened @quitterie-lcs #439

@wilbrdt wilbrdt mentioned this pull request Sep 19, 2023
13 tasks
Copy link
Contributor

@quitterie-lcs quitterie-lcs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's go !

Implemented camelCase aliases on API routes' params.
Ensures PEP8 compliance without disabling PyLint.
Implemented camelCase aliases on attributes of LRSStatementQuery.
Ensures PEP8 compliance without disabling PyLint.
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.

4 participants