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

Rename models and operations with mixed cases #904

Merged
merged 6 commits into from
Mar 25, 2022

Conversation

therve
Copy link
Contributor

@therve therve commented Mar 22, 2022

A few operations and models are affected by a small problem where a
character gets isolated. This fixes it.

@therve therve requested a review from a team as a code owner March 22, 2022 09:11
@therve therve changed the title Fix snake case issues Rename models and operations with mixed cases Mar 22, 2022
@therve therve force-pushed the therve/fix-snake-case-issues branch from 2180b5d to bb11a68 Compare March 23, 2022 13:20
@therve therve marked this pull request as draft March 24, 2022 08:26
@therve therve marked this pull request as ready for review March 24, 2022 08:26
A few operations and models are affected by a small problem where a
character gets isolated. This fixes it.
@therve therve force-pushed the therve/fix-snake-case-issues branch from bfcd099 to 2af84fe Compare March 24, 2022 08:40
tests/conftest.py Outdated Show resolved Hide resolved
@therve therve marked this pull request as draft March 25, 2022 07:43
@therve therve marked this pull request as ready for review March 25, 2022 07:43
@therve therve merged commit 7400fb8 into master Mar 25, 2022
@therve therve deleted the therve/fix-snake-case-issues branch March 25, 2022 08:14
github-actions bot pushed a commit that referenced this pull request Mar 25, 2022
* Fix snake case issues

A few operations and models are affected by a small problem where a
character gets isolated. This fixes it.

* Fix api as well

* Use a constant

* Fix NTML

* Extract to JSON file

* Imports 7400fb8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants