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

Modernize naïf -> naive #766

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

Modernize naïf -> naive #766

wants to merge 1 commit into from

Conversation

robinwhittleton
Copy link
Member

Also add a warning that naïf as a noun is the correct usage.

I’ve covered the rest of the corpus: the two remaining naïfs are part of French text.

Also add a warning that naïf as a noun is the correct usage.
@acabal
Copy link
Member

acabal commented Nov 4, 2024

In M-W, it's listed as naïf with the umlauts. So what we should do is normalize naif(s?) to naïf\1 in a separate regex instead of using the same one.

@robinwhittleton
Copy link
Member Author

All the naïf / naifs I’ve found in the corpus so far have been the adjective, which we said we’d modernize to naive. The warning is just so that people check that it’s not a noun.

Or am I misunderstanding?

@acabal
Copy link
Member

acabal commented Nov 4, 2024

Yes, but if naïf is a noun, and it occurs as naif, then we want to add the umalut in modernize-spelling.

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