Skip to content

chore(deps): update davidanson/markdownlint-cli2-action action to v22 (fix linter issues)#2243

Merged
frewilhelm merged 3 commits into
open-component-model:mainfrom
frewilhelm:fix-2221
Apr 10, 2026
Merged

chore(deps): update davidanson/markdownlint-cli2-action action to v22 (fix linter issues)#2243
frewilhelm merged 3 commits into
open-component-model:mainfrom
frewilhelm:fix-2221

Conversation

@frewilhelm

Copy link
Copy Markdown
Contributor

Summary

  • Updates DavidAnson/markdownlint-cli2-action from v20 to v22
  • Fixes 143 MD060 (table-column-style) violations across 14 website markdown files introduced by the new markdownlint v0.40.0 bundled in the action upgrade

What changed

The v22 action bundles markdownlint v0.40.0 which adds the MD060 rule, enforcing consistent table formatting. Tables were updated to use either compact style (| --- | --- |) or properly aligned columns depending on content length.

No content changes - only whitespace/formatting in markdown table syntax.

Test plan

  • npx markdownlint-cli2 passes with 0 errors across all 64 linted files
  • Verify website renders tables correctly in Netlify deploy preview

ocmbot Bot and others added 2 commits April 10, 2026 10:25
@netlify

netlify Bot commented Apr 10, 2026

Copy link
Copy Markdown

Deploy Preview for ocm-website ready!

Name Link
🔨 Latest commit 4b00581
🔍 Latest deploy log https://app.netlify.com/projects/ocm-website/deploys/69d8e7cd6a413e0008260cc0
😎 Deploy Preview https://deploy-preview-2243--ocm-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai

coderabbitai Bot commented Apr 10, 2026

Copy link
Copy Markdown
Contributor

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 77fcddff-752f-433e-a5de-463ccc8bcfe7

📥 Commits

Reviewing files that changed from the base of the PR and between a98538c and 4b00581.

📒 Files selected for processing (15)
  • .github/workflows/markdown.yml
  • website/README.md
  • website/content/blog/2026-03-16-ocm-controllers-differences.md
  • website/content/docs/concepts/component-identity.md
  • website/content/docs/concepts/signing-and-verification-concept.md
  • website/content/docs/how-to/configure-signing-credentials.md
  • website/content/docs/how-to/download-resources-from-component-versions.md
  • website/content/docs/how-to/generate-signing-keys.md
  • website/content/docs/how-to/verify-component-version.md
  • website/content/docs/overview/core-model.md
  • website/content/docs/reference/credential-consumer-identities.md
  • website/content/docs/reference/input-and-access-types.md
  • website/content/docs/reference/kubernetes-api/_index.md
  • website/content/docs/tutorials/credential-resolution.md
  • website/content_templates/template-concept.md

📝 Walkthrough

Walkthrough

Updated GitHub Actions markdown-lint workflow to version 22 and standardized Markdown table header separator formatting across multiple website documentation files by adding spacing around pipe characters.

Changes

Cohort / File(s) Summary
GitHub Workflow
.github/workflows/markdown.yml
Updated markdownlint-cli2-action from v20 to v22; no control flow or input changes.
Website Documentation
website/README.md, website/content/blog/2026-03-16-ocm-controllers-differences.md, website/content/docs/concepts/component-identity.md, website/content/docs/concepts/signing-and-verification-concept.md, website/content/docs/how-to/configure-signing-credentials.md, website/content/docs/how-to/download-resources-from-component-versions.md, website/content/docs/how-to/generate-signing-keys.md, website/content/docs/how-to/verify-component-version.md, website/content/docs/overview/core-model.md, website/content/docs/reference/credential-consumer-identities.md, website/content/docs/reference/input-and-access-types.md, website/content/docs/reference/kubernetes-api/_index.md, website/content/docs/tutorials/credential-resolution.md, website/content_templates/template-concept.md
Reformatted Markdown table header separators by standardizing spacing around pipe characters (e.g., |---|| --- |); no semantic or content changes.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

Suggested reviewers

  • Skarlso
  • morri-son

Poem

🐰 The tables now align with grace,
Pipes spaced with methodical pace,
From v20 up to v22 we spring,
Markdown formatting—such a simple thing!
Consistency reigns, both tool and design. 📋✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: updating markdownlint-cli2-action to v22 and fixing associated linter violations. It is clear, specific, and concise.
Description check ✅ Passed The description provides a clear, related explanation of the changes: dependency upgrade, linter rule (MD060) details, formatting fixes, and test plans. It is well-structured and directly relevant to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions Bot added the component/github-actions Changes on GitHub Actions or within `.github/` directory label Apr 10, 2026
@github-actions github-actions Bot added kind/chore chore, maintenance, etc. kind/dependency dependency update, etc. size/m Medium labels Apr 10, 2026
@frewilhelm frewilhelm marked this pull request as ready for review April 10, 2026 12:07
@frewilhelm frewilhelm requested a review from a team as a code owner April 10, 2026 12:07
@frewilhelm

Copy link
Copy Markdown
Contributor Author

I am not sure yet if we should just disable the rule. lets see how it works.

@frewilhelm frewilhelm merged commit ec65bb6 into open-component-model:main Apr 10, 2026
24 checks passed
ocmbot Bot pushed a commit that referenced this pull request Apr 10, 2026
… (fix linter issues) (#2243)

## Summary

- Updates `DavidAnson/markdownlint-cli2-action` from v20 to v22
- Fixes 143 MD060 (table-column-style) violations across 14 website
markdown files introduced by the new `markdownlint v0.40.0` bundled in
the action upgrade

## What changed

The v22 action bundles markdownlint v0.40.0 which adds the
[MD060](https://github.com/DavidAnson/markdownlint/blob/main/doc/md060.md)
rule, enforcing consistent table formatting. Tables were updated to use
either compact style (`| --- | --- |`) or properly aligned columns
depending on content length.

No content changes - only whitespace/formatting in markdown table
syntax.

## Test plan

- [x] `npx markdownlint-cli2` passes with 0 errors across all 64 linted
files
- [ ] Verify website renders tables correctly in Netlify deploy preview

---------

Signed-off-by: Frederic Wilhelm <frederic.wilhelm@sap.com>
Co-authored-by: ocmbot[bot] <125909804+ocmbot[bot]@users.noreply.github.com> ec65bb6
@frewilhelm frewilhelm deleted the fix-2221 branch April 10, 2026 12:15
morri-son pushed a commit to morri-son/open-component-model that referenced this pull request Apr 14, 2026
… (fix linter issues) (open-component-model#2243)

## Summary

- Updates `DavidAnson/markdownlint-cli2-action` from v20 to v22
- Fixes 143 MD060 (table-column-style) violations across 14 website
markdown files introduced by the new `markdownlint v0.40.0` bundled in
the action upgrade

## What changed

The v22 action bundles markdownlint v0.40.0 which adds the
[MD060](https://github.com/DavidAnson/markdownlint/blob/main/doc/md060.md)
rule, enforcing consistent table formatting. Tables were updated to use
either compact style (`| --- | --- |`) or properly aligned columns
depending on content length.

No content changes - only whitespace/formatting in markdown table
syntax.

## Test plan

- [x] `npx markdownlint-cli2` passes with 0 errors across all 64 linted
files
- [ ] Verify website renders tables correctly in Netlify deploy preview

---------

Signed-off-by: Frederic Wilhelm <frederic.wilhelm@sap.com>
Co-authored-by: ocmbot[bot] <125909804+ocmbot[bot]@users.noreply.github.com>

Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>
morri-son pushed a commit to morri-son/open-component-model that referenced this pull request Apr 15, 2026
… (fix linter issues) (open-component-model#2243)

## Summary

- Updates `DavidAnson/markdownlint-cli2-action` from v20 to v22
- Fixes 143 MD060 (table-column-style) violations across 14 website
markdown files introduced by the new `markdownlint v0.40.0` bundled in
the action upgrade

## What changed

The v22 action bundles markdownlint v0.40.0 which adds the
[MD060](https://github.com/DavidAnson/markdownlint/blob/main/doc/md060.md)
rule, enforcing consistent table formatting. Tables were updated to use
either compact style (`| --- | --- |`) or properly aligned columns
depending on content length.

No content changes - only whitespace/formatting in markdown table
syntax.

## Test plan

- [x] `npx markdownlint-cli2` passes with 0 errors across all 64 linted
files
- [ ] Verify website renders tables correctly in Netlify deploy preview

---------

Signed-off-by: Frederic Wilhelm <frederic.wilhelm@sap.com>
Co-authored-by: ocmbot[bot] <125909804+ocmbot[bot]@users.noreply.github.com>
Signed-off-by: Gerald Morrison (SAP) <gerald.morrison@sap.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/github-actions Changes on GitHub Actions or within `.github/` directory kind/chore chore, maintenance, etc. kind/dependency dependency update, etc. size/m Medium

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants