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

Documentation changes to .md files #816

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

AsherAli-SWE
Copy link

Changes I made and why

Added changes to markdown files to be more consistent with each other in terms of formatting. I followed the linting rules of markdownlint, a popular VS code plugin, and the formatting of Keep a Changelog to make these changes.

Majority of these changes had no visual impact on the rendered markdown, however they arguably improved code readability while also adding consistency to how they were written inside markdown.

Changes with visual impact:

  • added a shell language tag to a code fence in README.md to give small highlighting
  • removed semicolons in CHANGELOG.md that were inconsistently placed (i.e. Update: and Update)
  • changed every markdown file to have a top-level heading where applicable

Other possible changes to build on

CHANGELOG.md does not exactly follow Keep a Changelog, as its missing the "unreleased" section at the top as well as links on every version number showing the GitHub diff comparison between that version and the previous. This could also be done in the future if possible/appropriate.

Side Note

This is my first time making a contribution to a repository outside of academia and personal projects. Sorry in advance if I did anything incorrectly and please let me know what I should/could have done!

Changes were made to be consistent with rules set by popular markdown linting tools.

Specifically, rules in vs code plugin "markdownlint" by David Anson was used for the changes to fix violations of the following rules:

MD022
MD004
MD007
MD012
MD040
MD032
MD031

Rules found on the github repo here:

https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md
Changes were made to be consistent with rules set by popular markdown linting tools.

Specifically, rules in vs code plugin "markdownlint" by David Anson was used for the changes to fix violations of the following rules:

MD041
MD009
MD007


Rules found on the github repo here:

https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md
Changes were made to be consistent with rules set by popular markdown linting tools.

Specifically, rules in vs code plugin "markdownlint" by David Anson was used for the changes to fix violations of the following rules:

MD022
MD012
MD004
MD007
MD040
MD032
MD031

Although rule MD004 is a stylistic choice to alternate between symbols, it is inconsistent with the latest markdown document in the repository "CHANGELOG.MD" which uses hyphens for lists.

Rules listed above can be found here:

https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md
Changes were made to closer match the format of the markdown used by "Keep a Changelog".

Changes were made to remove inconsistencies between different sections of the changelog (i.e. Updated: and Updated)

Changes were made to be consistent with rules set by popular markdown linting tools.

Specifically, rules in vs code plugin "markdownlint" by David Anson was used for the changes to fix violations of the following rules:

MD022
MD032
MD026
Changed * list to - to be consistent with CHANGELOG.md and README.MD
@jdneo
Copy link
Member

jdneo commented Feb 1, 2024

Hi @AsherAtTrent,

Thank you for contribution. I'm ok for making changes to readme and changelog. Please not change the security.md because it's generated automatically from our org. If its content needs to be updated, some bots should help to do that I think.

@jdneo jdneo added the eng engineering label Feb 1, 2024
@Vstuelke
Copy link

gh pr checkout 816

Copy link

@Eziknomad Eziknomad left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

these are some log changes

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

Successfully merging this pull request may close these issues.

4 participants