Skip to content

Fix markdown-fail-fast on push#7057

Merged
dmathieu merged 4 commits intoopen-telemetry:mainfrom
MrAlias:markdownlint-push
Jul 22, 2025
Merged

Fix markdown-fail-fast on push#7057
dmathieu merged 4 commits intoopen-telemetry:mainfrom
MrAlias:markdownlint-push

Conversation

@MrAlias
Copy link
Copy Markdown
Contributor

@MrAlias MrAlias commented Jul 21, 2025

The recent fix for the markdown-fail-fast action fixed the action for PRs, but pushes to main are failing.

This fixes the action to run on pushes to main.

  • Match markdown files using action on syntax to skip unneeded runs
  • Parse event type in script
  • Handle push event types in script

@MrAlias MrAlias added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jul 21, 2025
@MrAlias MrAlias force-pushed the markdownlint-push branch from 24860bf to cc7418e Compare July 21, 2025 17:23
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.9%. Comparing base (784f637) to head (5859aa8).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff          @@
##            main   #7057   +/-   ##
=====================================
  Coverage   82.8%   82.9%           
=====================================
  Files        261     261           
  Lines      24371   24371           
=====================================
+ Hits       20201   20204    +3     
+ Misses      3795    3792    -3     
  Partials     375     375           

see 2 files with indirect coverage changes

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

@MrAlias
Copy link
Copy Markdown
Contributor Author

MrAlias commented Jul 21, 2025

@MrAlias MrAlias marked this pull request as ready for review July 21, 2025 17:33
@dmathieu
Copy link
Copy Markdown
Member

Merging this without waiting for 24h to fix the CI.

@dmathieu dmathieu merged commit 3e440ba into open-telemetry:main Jul 22, 2025
30 checks passed
@MrAlias MrAlias added this to the v1.38.0 milestone Aug 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip Changelog PRs that do not require a CHANGELOG.md entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants