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-10353. Add GitHub Actions check of all generated URLs in the sitemap. #77

Merged
merged 4 commits into from
Feb 22, 2024

Conversation

errose28
Copy link
Contributor

Summary

Format checking of file names (coming soon in HDDS-10349) will help ensure that URLs generated are in a consistent format, but it is still possible to define URLs through other means, like slug in the markdown frontmatter. We can catch these errors by checking the final sitemap.xml file that docusaurus generates as part of the build output.

Spelling will be enforced during static analysis as part of #76 since dev dependencies are not available during the build stage and docusaurus should not add new words to the URLs that were not already in the source files.

Jira

HDDS-10353

Testing

  • Workflow on this PR should pass.
  • Creating a URL with an underscore failed the check on my fork.

@github-actions github-actions bot added the website-v2 Tasks for the new ozone website targeting the HDDS-9225-website-v2 branch label Feb 21, 2024
@errose28 errose28 requested a review from adoroszlai February 21, 2024 00:26
.github/scripts/url.sh Outdated Show resolved Hide resolved
Co-authored-by: Doroszlai, Attila <[email protected]>
@errose28
Copy link
Contributor Author

Thanks for the review @adoroszlai. Typo is fixed.

@adoroszlai adoroszlai merged commit 3073baf into apache:HDDS-9225-website-v2 Feb 22, 2024
8 checks passed
@adoroszlai
Copy link
Contributor

Thanks @errose28 for the patch.

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.

2 participants