Skip to content

Trigger markdown lint on github action updates & fix rules #419

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

Merged
merged 1 commit into from
May 21, 2025

Conversation

ogenstad
Copy link
Contributor

@ogenstad ogenstad commented May 21, 2025

Fixes an issue with the CI where we upgraded markdown lint in #414, but the CI wasn't configured to run the markdown job during such an upgrade.

Now we see failures as there are violations based on the new rules from markdown lint. Which have to do with the text of anchor links.

The second update is to update the ruleset to ignore these new violations, we can go back later and re apply them again if desired.

@github-actions github-actions bot added the group/ci Issue related to the CI pipeline label May 21, 2025
Copy link

cloudflare-workers-and-pages bot commented May 21, 2025

Deploying infrahub-sdk-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: f6f7842
Status: ✅  Deploy successful!
Preview URL: https://c4053b09.infrahub-sdk-python.pages.dev
Branch Preview URL: https://pog-fix-markdownlint.infrahub-sdk-python.pages.dev

View logs

Copy link

codecov bot commented May 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

@@           Coverage Diff            @@
##           develop     #419   +/-   ##
========================================
  Coverage    75.16%   75.16%           
========================================
  Files          100      100           
  Lines         8613     8613           
  Branches      1689     1689           
========================================
  Hits          6474     6474           
  Misses        1676     1676           
  Partials       463      463           
Flag Coverage Δ
integration-tests 34.21% <ø> (+0.01%) ⬆️
python-3.10 47.23% <ø> (-0.03%) ⬇️
python-3.11 47.25% <ø> (+0.02%) ⬆️
python-3.12 47.20% <ø> (-0.03%) ⬇️
python-3.13 47.20% <ø> (-0.03%) ⬇️
python-3.9 45.56% <ø> (ø)
python-filler-3.12 24.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ogenstad ogenstad requested a review from a team May 21, 2025 11:16
@@ -11,3 +11,5 @@ MD034: false # no-bare-urls
MD041: false # allow 1st line to not be a top-level heading (required for Towncrier)
MD045: false # no alt text around images
MD047: false # single trailing newline
MD059: # Link descriptions that are provibited
Copy link
Contributor

Choose a reason for hiding this comment

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

prohibited I suppose?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

🤦

@ogenstad ogenstad force-pushed the pog-fix-markdownlint branch from d6ff058 to f6f7842 Compare May 21, 2025 12:59
@ogenstad ogenstad merged commit 95b2491 into develop May 21, 2025
18 checks passed
@ogenstad ogenstad deleted the pog-fix-markdownlint branch May 21, 2025 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
group/ci Issue related to the CI pipeline
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants