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

Fix singularization of words ending in ases #1524

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bisen2
Copy link
Contributor

@bisen2 bisen2 commented Mar 4, 2025

This PR would resolve #1508.

It would, however, be a breaking change. Any consumers currently using any *ases words as singular *asis would need to update their type names after updating this package.

I'm not sure if this is actually a change we would want to merge given the breaking nature, but figured I would open the PR and see what the maintainers thought.

@dsyme
Copy link
Contributor

dsyme commented Mar 11, 2025

@bisen2 I'm OK with this change. Could you add a major version bump as part of the change? Thanks

@bisen2
Copy link
Contributor Author

bisen2 commented Mar 13, 2025

Sure thing. Do you want me to mark it as a beta or just go straight for the 7.0.1, @dsyme?

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.

[JsonProvider] Plural word 'purchases' renamed as Purchasis in the .NET typespace
2 participants