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

docs(readme): fix typos and improve formatting #715

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

trueNAHO
Copy link
Contributor

@trueNAHO trueNAHO commented Jul 4, 2023

  • docs(readme): remove space between slash in Colemak/Dvorak
  • docs(readme): wrap text at 80 characters
  • style(readme): apply HTML LSP formatting
  • docs(readme): remove trailing list seperator
  • style(readme): properly format paragraphs
  • docs(readme): use are instead of is for plural references

@trueNAHO trueNAHO force-pushed the improve-readme branch 2 times, most recently from 6b9b13c to 11d8431 Compare July 4, 2023 17:23
@slotThe
Copy link
Member

slotThe commented Jul 9, 2023

Thanks; I think only 6cdf7cc and d4e7441 are non-stylistic in nature, and I don't think the others are necessarily "improvements" over the current state. Could you remove all commits but those two?

@trueNAHO
Copy link
Contributor Author

Thanks; I think only 6cdf7cc and d4e7441 are non-stylistic in nature, and I don't think the others are necessarily "improvements" over the current state. Could you remove all commits but those two?

Commits 11d8431, a5d45f2, and 7ef5d12 have been reverted. However, 6802212 was kept because it ensures proper verb agreement by using are instead of is in order to accurately convey that the vast capabilities (plural) are the distinguishing features of KMonad.

@slotThe slotThe merged commit 4b4670d into kmonad:master Jul 19, 2023
11 of 16 checks passed
@slotThe
Copy link
Member

slotThe commented Jul 19, 2023

Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants