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

HDDS-10254. Add GitHub Actions check for Markdown style. #81

Merged

Conversation

errose28
Copy link
Contributor

Summary

Markdown has many different ways to do the same thing, and some catches that can cause things to render in unexpected ways. This PR proposes the following changes:

  1. Add a markdownlint check to CI to enforce consistent formatting.
  2. Configure markdownlint rules for validating files.
  3. Fix existing markdown files to comply with the rules. Most of the fixes were done by passing the --fix flag to markdownlint.

While most of the configurations for markdownlint are set to prevent errors or enforce consistent formatting across documents, some of them, like line length, are more opinionated. I've set what I think are sensible values based on my experience but I'm open to other thoughts on these as well. If you are only interested in the formatting rules, see .markdownlint.yaml.

Jira

HDDS-10254

Testing

  • This run on my fork failed with just the linting check applied and no changes to existing markdown documents.
  • This run on my fork passed after fixing the markdown documents to comply with the linting rules.
  • CI on this PR should pass.

Conflicting files have been renamed/moved/deleted since the change was
originally done. They were ignored and not added as part of the cherry
pick

Conflicts:
docs/03-core-concepts/04-security/01-authentication/02-spnego.md
docs/03-core-concepts/04-security/01-authentication/04-s3-secrets.md
docs/05-administrator-guide/03-operations/02-upgrades-and-downgrades.md
docs/05-administrator-guide/03-operations/09-observability/03-component-web-uis.md
@github-actions github-actions bot added the website-v2 Tasks for the new ozone website targeting the HDDS-9225-website-v2 branch label Feb 27, 2024
Copy link

@swagle swagle left a comment

Choose a reason for hiding this comment

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

+1

@adoroszlai adoroszlai merged commit c3b3b3a into apache:HDDS-9225-website-v2 Mar 2, 2024
10 checks passed
@adoroszlai
Copy link
Contributor

Thanks @errose28 for the patch, @swagle for the review.

@errose28
Copy link
Contributor Author

errose28 commented Mar 4, 2024

Thanks @adoroszlai and @swagle for review and merge.

errose28 added a commit to errose28/ozone-site that referenced this pull request Mar 5, 2024
…ting-guide

* HDDS-9225-website-v2:
  HDDS-10254. Add GitHub Actions check for Markdown style. (apache#81)
errose28 added a commit to errose28/ozone-site that referenced this pull request Mar 5, 2024
* HDDS-9225-website-v2:
  HDDS-10254. Add GitHub Actions check for Markdown style. (apache#81)
  HDDS-10349. Add GitHub Actions check for consistent file name formatting. (apache#79)
  HDDS-10426. Crop ozone-logo.svg to a proper size (apache#80)
  HDDS-10353. Add GitHub Actions check of all generated URLs in the sitemap. (apache#77)
errose28 added a commit to errose28/ozone-site that referenced this pull request May 30, 2024
* HDDS-9225-website-v2:
  Bump docusaurus to 3.3.2 (apache#93)
  HDDS-10667. Improvements to spelling checks. (apache#89)
  HDDS-10698. Bump skywalking-eyes to v0.6.0 (apache#90)
  HDDS-10449. Add quick start to the top of contributing guide. (apache#83)
  HDDS-10351. Add GitHub Actions check for yaml formatting (apache#87)
  HDDS-9567. Add GitHub Actions license header check for relevant files. (apache#86)
  HDDS-10506. Reduce directory iterations in sidebar check (apache#85)
  HDDS-9866. Add GitHub Actions checks for consistent Docusaurus formatting. (apache#84)
  HDDS-10254. Add GitHub Actions check for Markdown style. (apache#81)
  HDDS-10349. Add GitHub Actions check for consistent file name formatting. (apache#79)
  HDDS-10426. Crop ozone-logo.svg to a proper size (apache#80)
  HDDS-10353. Add GitHub Actions check of all generated URLs in the sitemap. (apache#77)
  HDDS-9868. Add GitHub Actions check for spelling. (apache#76)
  HDDS-10400. Fix event condition in website publish workflow (apache#78)
  HDDS-10352. Add GitHub Actions workflow to build and run the website. (apache#74)
  HDDS-10222. Add pnpm guide to contributing guide. (apache#64)
  HDDS-10313. Update "Redundant" to "Reliable" in new website. (apache#73)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
website-v2 Tasks for the new ozone website targeting the HDDS-9225-website-v2 branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants