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

Fix various style issues and typos throughout the guide #147

Open
19 of 27 tasks
bredamc opened this issue Jan 5, 2022 · 47 comments · Fixed by #380
Open
19 of 27 tasks

Fix various style issues and typos throughout the guide #147

bredamc opened this issue Jan 5, 2022 · 47 comments · Fixed by #380
Assignees
Labels
General update General updates to the guide or repo

Comments

@bredamc
Copy link
Contributor

bredamc commented Jan 5, 2022

@ndeevy
Copy link
Contributor

ndeevy commented Jan 5, 2022

Looks like great improvements and suggestions @bredamc
Some product teams might like to weigh in on changes to their sections because conventions can vary depending on the products' upstream communities and how they like to do things.

@bredamc
Copy link
Contributor Author

bredamc commented Jan 5, 2022

Thanks @ndeevy!

Can you please advise which items should be referred for consultation, and suggest who should be consulted?

@ndeevy
Copy link
Contributor

ndeevy commented Jan 5, 2022

Sure :-]
Once the changes are in a PR we can see what's what, review with the larger group, and people can jump in if they have alternative suggestions from product areas too.

@bburt-rh bburt-rh added the General update General updates to the guide or repo label Jan 11, 2022
@bergerhoffer
Copy link
Collaborator

I updated the issue to add checkmarks (so that we can mark which ones have been taken care of vs. still need to be done) and also added a [Volunteer needed] spot on each, so that we can break these up and decide who can do which updates.

@bredamc were you interested in submitting a pull request for some of these items, or would you rather someone else on the team work on them?

I'd recommend to start on some of the ones that are pretty straightforward (such as 2, 3, 4, 6, 9, 11, 14, 20, 25, 26).

We will bring this up at the next SSG meeting (Feb 23) and look for volunteers to help with the whole list. Thanks again for doing this thorough review!

@bredamc
Copy link
Contributor Author

bredamc commented Feb 10, 2022

Thanks @bergerhoffer! I'm happy to work on them, now that I'm a little more proficient at working with Git forks :)
Should I wait until they are discussed at the meeting, or make some changes before then?

@bergerhoffer
Copy link
Collaborator

@bredamc It's up to you! I think that you could safely submit a PR for the numbers I mentioned above, and we can tackle the others in separate PRs, if they need additional discussion. Or if you'd prefer to wait until after the meeting, that's totally fine too.

@bredamc
Copy link
Contributor Author

bredamc commented Feb 22, 2022

@bergerhoffer I've made a start! Is there a way to tag the PR as "work in progress" until it's ready for review?
#163

@bergerhoffer
Copy link
Collaborator

@bredamc Awesome! And yes, you can add a [WIP] to the beginning of the PR title. If you can add labels, you could also add the "On hold" label.

@bergerhoffer
Copy link
Collaborator

@bredamc I've taken a closer look at the rest and here are my thoughts on the ones that are left:

  • (1) Yes, we definitely don't want to be duplicating terms that are already covered in the ISG (unless we are deviating from the suggested guidance). +1 for removing the duplicate terms you identified
  • (5) I don't think I follow what you're recommending about KB/MB/GB being inconsistent. But I say to go ahead with the suggested change in a PR and we can review. I'd also be okay with you making offline backup and online backup more parallel.
  • (7) These examples have recently been revamped. Can you take another look to see if there are still any lingering issues? And if so, feel free to make the suggested changes in a PR
  • (8) +1 for 8a. +1 for 8b if you're able to come up with a suggestion. If not, we can ping the team to figure it out.
  • (10) +1 I think we could just remove "because" all together here since (:wink:) "since" is in the ISG
  • (12) I honestly don't know what to do about this one 😄
  • (13) I think you did this one already
  • (15) I think we could safely remove "diskettes" from the list here, since it's just an example.
  • (16) I'd be okay with however you want to update to make them more consistent. This one is a little rough to read in such a large block, but we can't use bullets/formatting in these descriptions to break it up nicer. Though honestly I'd be curious to know whether any of this actually deviates from what the ISG recommends. If not, we could consider pulling it completely.
  • (17) I think we could just remove the "See" sentence
  • (18) I feel like we could remove this. I think that if pop-up is going to be used, then it should be qualified as a pop-up window or pop-up list as the ISG recommends. (Similar to drop-down)
  • (19) I can't tell for sure, but it seems like this entry is implying that the enter key is actually listed lowercase "enter" on a Mac. I'd have to understand a little more to be able to make a recommendation.
  • (21) I'm fine with your reword suggestion
  • (22) +1
  • (23) +1
  • (24) If we wanted to change it to with caution, I think we'd need to specify in the description why. Feel free to make a suggestion here
  • (27) Possibly, but I'm unsure how it would look once removing.

@ndeevy
Copy link
Contributor

ndeevy commented Apr 27, 2022

I agree with all your suggested changes Breda, and Andrea's assessments. Kudos and thanks :-]

One thought about (27), we could just cut off the 'recommend' like:
Do not use "we suggest". Use a more direct construction or use "recommend". For example, instead of "We suggest that you make a backup of your data disk", write "Back up your data disk".

'Recommend' is kind of a tricky in itself, I see it come up in our docs a lot, but IBM doesn't speak directly to it alone -do you both think it could warrant its own entry?

@bredamc
Copy link
Contributor Author

bredamc commented Apr 27, 2022

Thank you, Andrea and Naomi!

I've reviewed your suggestions, Andrea:
(7). I've removed 7(c) ("In the "Deprecated functionality" section, is it correct to say "Red Hat do not implement" (two occurrences), or should it be "Red Hat does not implement"? "), but 7(a) and 7(b) are still valid.
(12) I can create a PR with a suggestion, but I wonder why these sections are included at all? Are these terms used in RH docs? Should we maybe request the addition of these terms to IBM Style instead?
(13) I did indeed do this one already! Apologies for not updating it at the time, I've updated it now.
(16) I'll review again to see if any of the advice is different from that in IBM Style.
(19) I think the IBM Style advice is to use an initial capital for the key name, regardless of the hardware used. The "Keyboard keys" topic includes the following Mac example: "Press Return."

I agree with your other recommendations, and am happy to work on them. Do we need to notify the Council about the sections that we propose to remove, or is it time enough to notify when requesting PR approval?

@bergerhoffer
Copy link
Collaborator

@bredamc Responses inline!

(12) I can create a PR with a suggestion, but I wonder why these sections are included at all? Are these terms used in RH docs? Should we maybe request the addition of these terms to IBM Style instead?

Good question. I don't know enough about these terms to know whether/where in RH docs they are used. But if they are included here, I would assume that someone had a need for them. But I would say that the best outcome would be for these terms to be in the ISG, if we want to ask them.

(19) I think the IBM Style advice is to use an initial capital for the key name, regardless of the hardware used. The "Keyboard keys" topic includes the following Mac example: "Press Return."

I would +1 the standardization of capitalizing the keys (especially since we don't always know what OS people are on). I'd say that we could probably remove both the "enter" and "return" entries, in favor of following the ISG.

I agree with your other recommendations, and am happy to work on them. Do we need to notify the Council about the sections that we propose to remove, or is it time enough to notify when requesting PR approval?

I think that requesting review/approval from the council during the PR review should be fine!

@bburt-rh
Copy link
Contributor

@bredamc When you get a chance, could you please update the status of this issue?

@bredamc
Copy link
Contributor Author

bredamc commented Sep 26, 2022

@bburt-rh Apologies for the delay :(
I will work on this issue when the master branch has been renamed to main.

@bergerhoffer
Copy link
Collaborator

@bredamc The branch rename is now complete, so you can feel free to resume this when you're ready. Thanks!

@dfitzmau
Copy link
Contributor

Hi @bredamc . Just checking in with you on this Issue. How is the progress? A mammoth task, so thanks for taking this Issue on!

@bredamc
Copy link
Contributor Author

bredamc commented Feb 21, 2023

Thanks for the reminder, @dfitzmau. I'd completed all of the subitems that I'd originally signed up for, but now that I know the content a little better, I'll review the other subitems to see if I can do a few more!

@Srivaralakshmi
Copy link
Contributor

@bredamc That's great and thank you so much for all the involvement. Deeply appreciate it!

Any updates about the progress on this issue?

@Srivaralakshmi
Copy link
Contributor

@bredamc That's great and thank you so much for all the involvement. Deeply appreciate it!

Any updates about the progress on this issue?

@bredamc Please respond. Gentle nudge : )

@bredamc
Copy link
Contributor Author

bredamc commented Mar 24, 2023

No update @Srivaralakshmi -- still on my to-do list!

@mportman12
Copy link
Collaborator

@bredamc any updates on this?

@CBID2
Copy link
Contributor

CBID2 commented Jul 30, 2023

Hi @bergerhoffer! :) I see no one has done the first checkmark, so I'd like to work on it.

@bergerhoffer
Copy link
Collaborator

Hi @CBID2, that would be great! You can just delete those 6 entries that are listed. I've just confirmed that they already exist in the IBM style guide, so we do not need to list them here in ours.

We have information on how to contribute and open a pull request here: https://github.com/redhat-documentation/supplementary-style-guide/blob/main/CONTRIBUTING.md

Please let us know if you have any questions!

@CBID2
Copy link
Contributor

CBID2 commented Aug 1, 2023

Hi @CBID2, that would be great! You can just delete those 6 entries that are listed. I've just confirmed that they already exist in the IBM style guide, so we do not need to list them here in ours.

We have information on how to contribute and open a pull request here: https://github.com/redhat-documentation/supplementary-style-guide/blob/main/CONTRIBUTING.md

Please let us know if you have any questions!

Great

@CBID2
Copy link
Contributor

CBID2 commented Aug 1, 2023

To clarify, do you want me to delete the files mentioned in checkboxes 1-6 @bergerhoffer?

@bergerhoffer
Copy link
Collaborator

@CBID2 Not the whole files (because the files are done by letter, so all "A" terms are in a.adoc). But for each of those entries, you would go into the appropriate letter file and delete the lines for the entry.

For example, for "agnostic", you'd go into a.adoc and remove these lines:

[[agnostic]]
==== image:images/no.png[no] agnostic (adjective)
*Description*: _Agnostic_ denotes or relates to hardware or software that is compatible with many types of platforms or operating systems. For example, many common file formats (JPEG, MP3, and others) are platform agnostic. Use "neutral" instead.

*Use it*: no

*Incorrect forms*:

*See also*:

@CBID2
Copy link
Contributor

CBID2 commented Aug 2, 2023

@CBID2 Not the whole files (because the files are done by letter, so all "A" terms are in a.adoc). But for each of those entries, you would go into the appropriate letter file and delete the lines for the entry.

For example, for "agnostic", you'd go into a.adoc and remove these lines:


[[agnostic]]

==== image:images/no.png[no] agnostic (adjective)

*Description*: _Agnostic_ denotes or relates to hardware or software that is compatible with many types of platforms or operating systems. For example, many common file formats (JPEG, MP3, and others) are platform agnostic. Use "neutral" instead.



*Use it*: no



*Incorrect forms*:



*See also*:

Oh I see. Thanks

@bergerhoffer
Copy link
Collaborator

Reopening because the above PR only addressed 8a in the list

@bergerhoffer bergerhoffer reopened this Aug 17, 2023
@CBID2
Copy link
Contributor

CBID2 commented Aug 17, 2023

Reopening because the above PR only addressed 8a in the list

I wonder how to solve 8b @bergerhoffer 🤔

@bergerhoffer
Copy link
Collaborator

I wonder how to solve 8b @bergerhoffer 🤔

We'd need to either clarify "this", or rewrite the sentence. We could do something like the change below to avoid using "this":

If your Apache web server configuration enables SSL security, verify that you enable only the TLSv1 protocol and disable SSLv2 and SSLv3 . This is because of to avoid the POODLE SSL vulnerability (CVE-2014-3566).

So the source for that would be:

If your Apache web server configuration enables SSL security, enable only the TLSv1 protocol and disable SSLv2 and SSLv3 to avoid the link:https://access.redhat.com/solutions/1232413[POODLE SSL vulnerability (CVE-2014-3566)].

@michellemacrh
Copy link
Contributor

Hello 👋 is this issue still open/active?

I'm happy to take a look at 22 (Shadowman) and 23 (spec file) if they're still relevant.

@CBID2
Copy link
Contributor

CBID2 commented Aug 31, 2023

I'll do number 21 @bergerhoffer

@CBID2
Copy link
Contributor

CBID2 commented Sep 7, 2023

Hey @bergerhoffer. I did the PR for number #21. It's #391.

@dfitzmau dfitzmau mentioned this issue Sep 7, 2023
@dfitzmau
Copy link
Contributor

dfitzmau commented Sep 7, 2023

Hi @redhat-documentation/ccs-style-council . I picked up item #12 and created the following PR that is ready for review:

#392

Recent IBM Cloud updates were made to the guide: #359 . The request for a usable IBM eServer System i term is no longer valid.

@bergerhoffer
Copy link
Collaborator

@michellemacrh yes, items 22 and 23 are still applicable if you want to create a PR for those. Thank you!

@michellemacrh
Copy link
Contributor

Hi @redhat-documentation/ccs-style-council.

I picked up item 22 and created the following PR that's ready for review: #398

I updated the link and the link title per the request.

@CBID2
Copy link
Contributor

CBID2 commented Sep 27, 2023

Hi @bergerhoffer! Item 21 is done, so now I'll do 10

@CBID2
Copy link
Contributor

CBID2 commented Sep 27, 2023

As promised @bergerhoffer, Item 10 is done. The PR is #406

@bredamc
Copy link
Contributor Author

bredamc commented Sep 28, 2023

I think it would be better to remove the entry altogether -- the IBM Style Guide already has an entry for "since" so we don't need the duplicated entry here. But I'll leave it to @bergerhoffer to advise :)

@dfitzmau dfitzmau mentioned this issue Sep 28, 2023
@CBID2
Copy link
Contributor

CBID2 commented Oct 4, 2023

Box 10 has been addressed with this PR @bredamc

@bredamc
Copy link
Contributor Author

bredamc commented Oct 4, 2023

@CBID2 Thank you :)

@Srivaralakshmi
Copy link
Contributor

@bredamc and @redhat-documentation/ccs-style-council Where does this one stand at the moment?

@bburt-rh
Copy link
Contributor

@bredamc What's the status of the remaining items for this issue?

@bergerhoffer
Copy link
Collaborator

Chatted with @bredamc and she will take a look at the items left after Summit, to see if there are any others we should prioritize addressing, or if we can close out this issue.

@CBID2
Copy link
Contributor

CBID2 commented Jun 5, 2024

Hey @bergerhoffer and @bredamc! :) I did a pull request for item #27. It's #486. Check it out.

@bredamc
Copy link
Contributor Author

bredamc commented Jun 13, 2024

Hey @bergerhoffer and @bredamc! :) I did a pull request for item #27. It's #486. Check it out.

@CBID2 Thank you for your PR but I think you misunderstood the original request (my fault for not spelling it out). As you can see in this comment, the original text for the "we suggest" entry mentioned using "recommend" instead. In item 27, I was requesting the removal of that text from the "we suggest" entry, and that change has since been made. I'll update item 27 to show that it's already done.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
General update General updates to the guide or repo
Projects
None yet
9 participants