Skip to content

Conversation

@bearomorphism
Copy link
Collaborator

@bearomorphismbearomorphism commented Dec 6, 2025

Let's try this.

Closes#1691

@codecov
Copy link

codecovbot commented Dec 6, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.88%. Comparing base (120d514) to head (ace981a).
⚠️ Report is 886 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #1696 +/- ## ========================================== + Coverage 97.33% 97.88% +0.55%  ========================================== Files 42 60 +18 Lines 2104 2605 +501 ========================================== + Hits 2048 2550 +502 + Misses 56 55 -1 
FlagCoverage Δ
unittests97.88% <ø> (+0.55%)⬆️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@bearomorphismbearomorphismforce-pushed the link-action branch 5 times, most recently from b95bbe6 to 59eb5bfCompareDecember 6, 2025 13:29
@bearomorphism
Copy link
CollaboratorAuthor

@woile@Lee-W would you like to create issue form when broken link is found?

If we want to automatically create issues for broken links, we may need to adjust our repo settings

Failed run: https://github.com/commitizen-tools/commitizen/actions/runs/19989102721/job/57327236001
Potential solution: https://stackoverflow.com/questions/70435286/resource-not-accessible-by-integration-on-github-post-repos-owner-repo-ac

@bearomorphismbearomorphismforce-pushed the link-action branch 10 times, most recently from 651fb63 to aaa410bCompareDecember 6, 2025 14:16
args: . --exclude-path "tests"
fail: false

# - name: Create or Update PR Comment
Copy link
CollaboratorAuthor

Choose a reason for hiding this comment

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

TBD

Copy link
Member

Choose a reason for hiding this comment

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

probbaly use it to create an issue instead of a PR?

Copy link
Member

Choose a reason for hiding this comment

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

agreed, let's start with an issue

Copy link
CollaboratorAuthor

Choose a reason for hiding this comment

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

Actually, I was thinking about the behavior of the pipeline when the link checker fails (non-empty error list) on a documentation PR.

args: . --exclude-path "tests"
fail: false

# - name: Create or Update PR Comment
Copy link
Member

Choose a reason for hiding this comment

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

probbaly use it to create an issue instead of a PR?

@bearomorphismbearomorphismforce-pushed the link-action branch 4 times, most recently from 6368980 to 7e5d8aeCompareDecember 14, 2025 03:20
@bearomorphism
Copy link
CollaboratorAuthor

@Lee-W@woile now this PR is ready. Could you take a look again? Thanks

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automate check for commitizen document broken links

3 participants

@bearomorphism@woile@Lee-W