Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

fix: update pluralization of yaml to non-disputable version #2876

Merged
merged 1 commit into from
Feb 26, 2020
Merged

fix: update pluralization of yaml to non-disputable version #2876

merged 1 commit into from
Feb 26, 2020

Conversation

morremeyer
Copy link
Contributor

Changelog version

This PR updates the pluralization of YAML in an error message from „yamels“ to „yaml files“.

Long version

We were debugging an issue with failing git repository clones today. While doing so, the pluralization of „yaml“ to „yamels“ was bugging me extremely, so I thought about a version of it that was possibly agreeable to more people.

Therefore, I changed it to „yaml files“.

@hiddeco hiddeco added UX In pursuit of a delightful user experience blocked-decision yolo Why not? Life's too short labels Feb 25, 2020
@shibumi
Copy link
Contributor

shibumi commented Feb 25, 2020

This PR erases signature history

Well, everything beautiful needs a point where it starts. I would appreciate the term yaml files instead of yamels.

Why?

  • When yaml files is used, it's clear that both parties (if human or not) talk about yaml
  • yamels is confusing, when I had a look on this term the first time, I thought it's a new fancy file format or whatever (not kidding).

@hiddeco
Copy link
Member

hiddeco commented Feb 25, 2020

No worries, I do 100% understand the PR from a UX perspective.

Having known @squaremo for a while, I know both yamel and yamels are persistent in his vocabulary. I actually had a good laugh while Googling for "yamels" as squaremo/kubeyaml (the tool we use for replacing i.e. image tags while making image updates) is the only YAML related result on the first page.

@squaremo
Copy link
Member

I bless this retro-corrective to my vocabulary

@hiddeco hiddeco removed blocked-decision yolo Why not? Life's too short labels Feb 26, 2020
@hiddeco hiddeco merged commit 8b3eb73 into fluxcd:master Feb 26, 2020
@hiddeco hiddeco added this to the 1.19.0 milestone Feb 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
UX In pursuit of a delightful user experience
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants