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

adds terraform misspellings to dictionary #1999

Merged

Conversation

apoclyps
Copy link
Contributor

What's Changed

Adds misspellings for terraform to the dictonary

@luzpaz
Copy link
Collaborator

luzpaz commented Aug 5, 2021

@apoclyps care to add the plural forms of each word + rebase? thx

@luzpaz luzpaz added the dictionary Changes to the dictionary label Aug 5, 2021
@luzpaz
Copy link
Collaborator

luzpaz commented Aug 30, 2021

@bl-ue i don't have permission access to push changes to a PR. Do you mind updating that?

@vikivivi
Copy link
Contributor

vikivivi commented Aug 31, 2021

@luzpaz In my personal opinion, terraform is a name of a software. Should be similar to the name Linux software without plural.

@apoclyps apoclyps force-pushed the add-terraform-to-dictonary branch from 4615882 to 0fe9acf Compare August 31, 2021 08:33
@apoclyps
Copy link
Contributor Author

@luzpaz updated to include the pural when the plural form is the intent e.g. teraforms to terraforms

@luzpaz
Copy link
Collaborator

luzpaz commented Aug 31, 2021

In my personal opinion, terraform is a name of a software. Should be similar to the name Linux software without plural.

It's a valid word as well.
Terraform (verb) to alter the environment of (a celestial body) in order to make capable of supporting terrestrial life forms.

@bl-ue
Copy link
Contributor

bl-ue commented Aug 31, 2021

@bl-ue i don't have permission access to push changes to a PR. Do you mind updating that?

@luzpaz I believe that's @apoclyps's responsibility, see here

@apoclyps
Copy link
Contributor Author

Allow edits from maintainers is already enabled on this pull request.

Screenshot 2021-08-31 at 16 33 03

@apoclyps apoclyps force-pushed the add-terraform-to-dictonary branch from 0fe9acf to 8907954 Compare August 31, 2021 15:35
@apoclyps apoclyps force-pushed the add-terraform-to-dictonary branch from 8907954 to 0cb27d5 Compare September 9, 2021 08:25
@apoclyps
Copy link
Contributor Author

apoclyps commented Sep 9, 2021

pull request updated and ammended by running make sort-dictionaries to sort the plural additions.

@luzpaz luzpaz merged commit 585ba06 into codespell-project:master Sep 9, 2021
@apoclyps apoclyps deleted the add-terraform-to-dictonary branch September 9, 2021 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dictionary Changes to the dictionary
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants