Skip to content

Core: new "AllowInfix" and "AllUppercase"/"AllLowercase" enum members #762

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

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

knocte
Copy link
Collaborator

@knocte knocte commented Aug 12, 2025

These allows people to use MACRO_CASE or snake_case in their naming configurations.

@knocte
Copy link
Collaborator Author

knocte commented Aug 13, 2025

@webwarrior-ws please review

Copy link
Contributor

@webwarrior-ws webwarrior-ws left a comment

Choose a reason for hiding this comment

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

Otherwise LGTM

* Some dots were missing at the end of sentences/paragraphs.
* Using colon and a list of elements rather than parentheses.
@knocte knocte force-pushed the wip/moreNamingConfigurations branch 4 times, most recently from 53b1b1e to cd8ce66 Compare August 14, 2025 18:57
knocte and others added 8 commits August 14, 2025 21:26
This paves the way for allowing people to use MACRO_CASE
or snake_case in their configurations.
Added another case to getLiteralIdents because F# compiler can
produce different AST for similar code.
Also modified a test to make sure they work.

Co-authored-by: webwarrior-ws <reg@webwarrior.ws>
In the case the user wants quick fixes to convert {camel,pascal}case
to snake_case or MACRO_CASE, we need to inject underscores when
case changes. This commit makes the previous added test pass.
Regarding splitByCaseChange: a more complex case.
@knocte knocte force-pushed the wip/moreNamingConfigurations branch from 7daa474 to dc8a79a Compare August 14, 2025 19:26
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.

2 participants