Skip to content

Conversation

eksperimental
Copy link
Contributor

No description provided.

@eksperimental eksperimental changed the title Replace mispelt helper nillify with nilify Replace misspelt helper nillify with nilify Jun 11, 2022
@josevalim
Copy link
Member

I actually have found no evidence one form is preferred than the other, as it is not an existing word. Have you found anything? :)

@eksperimental
Copy link
Contributor Author

eksperimental commented Jun 11, 2022

I actually have found no evidence one form is preferred than the other, as it is not an existing word. Have you found anything? :)

Well, I am not aware of a rule in English where you double up a letter to make a noun into a verb with the "ify" suffix.
I think the confusion may come from the word nullify which pretty much does the same, but null ends in double L.

https://en.wiktionary.org/wiki/villify
see it is mentioned as a misspelling from "vilify"

Ecto uses :nilify_all

@josevalim josevalim merged commit 27a276e into elixir-lang:main Jun 11, 2022
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@eksperimental eksperimental deleted the nilify branch June 11, 2022 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants