Skip to content

feat: allow specifying additional-initialisms #1733

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.

Sign up for GitHub

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

Jump to bottom
Merged
jamietanna merged 9 commits into from
Nov 28, 2024
Merged

feat: allow specifying additional-initialisms #1733

jamietanna merged 9 commits into from
Nov 28, 2024

Conversation

@micaelmalta
Copy link
Contributor

@micaelmalta micaelmalta commented Aug 21, 2024 •
edited
Loading

The goal of this PR is to be able to provide additional Initialisms to the generator.
Initialism like SSN or PTO should not be in camel case for example

How to use:

output-options:
  skip-prune: true
  name-normalizer: ToCamelCaseWithInitialisms
  additional-initialisms: 
    - NAME

@micaelmalta micaelmalta force-pushed the mickey/initialisms_override branch from ed1c9ad to 8fdbe7a Compare August 21, 2024 19:55
@micaelmalta micaelmalta changed the title Mickey/initialisms override Feat: additional initialisms Aug 21, 2024
@micaelmalta micaelmalta force-pushed the mickey/initialisms_override branch from 8fdbe7a to 493cb9c Compare August 21, 2024 19:57
ccoVeille
ccoVeille reviewed Aug 21, 2024
View reviewed changes
@micaelmalta micaelmalta force-pushed the mickey/initialisms_override branch 2 times, most recently from 414cc01 to 0ae10ae Compare August 21, 2024 20:38
@micaelmalta micaelmalta marked this pull request as ready for review August 21, 2024 20:47
@micaelmalta micaelmalta requested a review from a team as a code owner August 21, 2024 20:47
@cnnrrss
Copy link

cnnrrss commented Aug 21, 2024

Great suggestion @micaelmalta. I've also been looking for this feature.

@micaelmalta micaelmalta force-pushed the mickey/initialisms_override branch from 0ae10ae to c574ddc Compare August 21, 2024 20:55
@micaelmalta micaelmalta requested a review from ccoVeille August 21, 2024 20:58
@micaelmalta
Copy link
Contributor Author

micaelmalta commented Oct 1, 2024

Ping

@jamietanna jamietanna added this to the v2.5.0 milestone Oct 1, 2024
@jamietanna jamietanna changed the title Feat: additional initialisms feat: allow specifying additional-initialisms Nov 28, 2024
Jamie Tanna added 2 commits November 28, 2024 12:40
@jamietanna jamietanna enabled auto-merge (squash) November 28, 2024 13:07
@jamietanna jamietanna added the enhancement New feature or request label Nov 28, 2024
@jamietanna jamietanna merged commit b07f7ea into oapi-codegen:main Nov 28, 2024
19 checks passed
@jamietanna jamietanna mentioned this pull request Feb 12, 2025
Merged
@jamietanna jamietanna added the notable changes Used for release notes to highlight these more highly label May 11, 2025
@renovate renovate bot mentioned this pull request Jul 30, 2025
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jamietanna jamietanna jamietanna approved these changes

+1 more reviewer

@ccoVeille ccoVeille ccoVeille approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

enhancement New feature or request notable changes Used for release notes to highlight these more highly

Projects

None yet

Milestone

v2.5.0

Development

Successfully merging this pull request may close these issues.

4 participants

@micaelmalta @cnnrrss @jamietanna @ccoVeille