-
Notifications
You must be signed in to change notification settings - Fork 523
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
Update style guide with preferred terminology #152
Comments
Quick brain dump. I'll elaborate here later or in the next DAWGs meeting.
Ansible Lint or
Molecule or
Sure.
I don't think we need title case here. |
Updated the hackmd accordingly, thanks! The one thing I put there is the firs time someone talks about something (e.x. molecule) it should be Ansible Molecule. After that, we can drop the Ansible part except for Ansible Lint imo. |
I'd say we should always call it Ansible Navigator or On the other hand, I don't think we need to say Ansible Molecule (just Molecule or |
I'd also figure out how we want to name/differentiate ansible-core and the ansible package. Sometimes people use upper case Ansible which can be ambiguous. |
Looks really good. For readability it should probably be Ansible navigator and shorthand Navigator since that then substitutes the name of the tool. |
I think "execution environment" should follow the same rule of playbooks:
Also, not native speaker, but is "occurance" correct? Shouldn't it be "occurrence"? (this comment is valid for the whole document) |
Also:
Should be
|
thanks all. I've updated the hackmd. |
@samccann we might want to add the following to the list and keep them consistent with Ansible Navigator:
Also, we might want to keep the Caps for subsequent mentions to avoid confusion between referring to the command/tool vs a normal word in a blog post for ex. (Navigator, Builder, Runner, etc.) |
Thanks @leogallego - The caps aren't necessary in that regard because the cli commands should always be formated as |
I don't think we should encourage EE to refer to execution environments, if we opt for uncapitalized form, since it would be weird to do so |
According to http://docs.testing.ansible.com/ansible/devel/dev_guide/style_guide/spelling_word_choice.html#acronyms it should be EE for execution environment(s). |
An acronym is an acronym if a group of people decides to use it in that way. My point is not to suggest a different acronym (eg: ee), but to avoid suggesting an acronym at all, in the same way that we do not use AP for Ansible playbooks, etc |
+1 for avoiding suggesting EE. It's a good opportunity to come to a consensus on this one as it is gaining acceptance. I feel like the whole purpose of EE is to have a shorthand for the writer because using "execution environment" repeatedly is kind of a chore. As a reader, though, I think EE as a standalone acronym is slightly confusing. My preference for the guidance here would be something like use "execution environment" in reference to the abstract notion and then a format like " |
Also "EE", for many tech people, is associated to Java (J2EE) and stands for Enterprise Edition. |
Why never Ansible core? |
This was my attempt to clear up the confusion with all things named Ansible. I figured if in the docs, we never say Ansible core, but always |
Hey @samccann to give some context, I'm talking about capitalization as a means of distinguishing what are looking more and more like actions / common words from the actual tools / applications, not for the CLI command itself, but when talking about it. For ex. talking about Execution Environments, or Builder, or any other tool without capitalizing makes it harder to distinguish the use depending on the context. Several of the projects are being named after common words (Lint, Navigator, Builder, Sign, Runner, Receptor... etc.), capitalization helps to quickly figure if we are talking about the tool or not, even if we are not referring to the CLI to run a command. This also helps international users, as some might have a hard time some times using translation tools to figure if something is an action or a tool when checking documentation. We have run into translated project names several times (even within Ansible projects user interfaces), using Caps is a clear sign that's not a simple word. In my opinion we should stick to capitals if we are not using the CLI command, I see it as an accessibility issue.
@oraNod And the above about keeping Caps would make using EE valid again after a full first "Execution Environment (EE)" in the text. I agree it might be slightly confusing on first sight if there is no full name before, but people tend to use acronyms for a reason and it's the same we use automation: we are lazy :D |
My suggestion would be:
Examples:
Speaking about Ansible as a special case. If the word order I think i'm generally agree with what @leogallego is saying above. |
By the same token I feel like if you want Ansible to only refer to the package then you should only use |
I also generally agree with @leogallego and @Andersson007 but still don't think "EE" should ever be standalone in a sentence. It's not a hill I want to die on though. What would be great here is a visualization of all these things with the naming conventions. Sort of like the Fedora org chart but for the Ansible ecosystem. |
@oraNod - in the rst, we used a variable (|ee| = execution environment) to make it convenient for writers to not have to spell it all out. Could we do the same int his context? |
@tvo318 I suppose we could do that but I'm not sure if that adds to complexity for contributors. If we did, I'd prefer to put any of those variables somewhere other than the |
not to bikeshed, maybe we should submit a PR reflecting the suggested options based on thumbs up or whatever? We could skip EE for now. |
Good idea. I'll open a PR with the contents of the hackmd and see how it goes :-) |
To ensure we are consistent in naming projects and what not, we should define our common terms and add them to the docs style guide.
This is a scratchpad where we can decide the terms and then someone can create the PR as an good first issue.
https://hackmd.io/SH4UvQCHS4OcRsRY5qLi0A
The text was updated successfully, but these errors were encountered: