Skip to content

Conversation

@bearomorphism
Copy link
Collaborator

@bearomorphism bearomorphism commented Dec 6, 2025

Let's try this.

Closes #1691

@codecov
Copy link

codecov bot commented Dec 6, 2025

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1696      +/-   ##
==========================================
+ Coverage   97.33%   98.68%   +1.34%     
==========================================
  Files          42       60      +18     
  Lines        2104     2656     +552     
==========================================
+ Hits         2048     2621     +573     
+ Misses         56       35      -21     
Flag Coverage Δ
unittests 98.68% <ø> (+1.34%) ⬆️

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.

@bearomorphism bearomorphism force-pushed the link-action branch 5 times, most recently from b95bbe6 to 59eb5bf Compare December 6, 2025 13:29
@bearomorphism
Copy link
Collaborator Author

@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

@bearomorphism bearomorphism force-pushed the link-action branch 9 times, most recently from 3ecbb5c to 651fb63 Compare December 6, 2025 14:14
args: . --exclude-path "tests"
fail: false

# - name: Create or Update PR Comment
Copy link
Collaborator Author

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

- cron: "00 18 * * *"

jobs:
linkChecker:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
linkChecker:
check-links:

for consistency

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?

Sign up for free to 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