Skip to content

Conversation

@andyjakubowski
Copy link
Contributor

@andyjakubowski andyjakubowski commented Oct 17, 2025

The Check Links CI job is flaky because of 429 Too Many Requests from YouTube. We can safely ignore that link and make the CI job more stable.

Signed-off-by: Andy Jakubowski hello@andyjakubowski.com

Summary by CodeRabbit

  • Chores
    • Updated build workflow configuration to ignore a YouTube URL in link verification checks.

Note: This release contains internal configuration updates with no user-facing changes.

Signed-off-by: Andy Jakubowski <hello@andyjakubowski.com>
@linear
Copy link

linear bot commented Oct 17, 2025

GRN-4962 Ignore YouTube link in CI job

The Check Links CI job is flaky because of 429 Too Many Requests from YouTube. We can safely ignore that link and make the CI job more stable.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 17, 2025

📝 Walkthrough

Walkthrough

Updates to .github/workflows/build.yml extend the ignore_links string in the check-links steps by adding a YouTube URL (https://www.youtube.com/watch?v=jF0yA-JLQW0). The URL is added in two locations: within the build workflow's Check Links step and within a separate check_links job. No changes to logic or public APIs.

Pre-merge checks

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title accurately summarizes the main change: adding a YouTube URL to the ignore_links configuration in the check-links CI job. It is specific, concise (50 characters), and uses conventional commit formatting. A teammate scanning the history would immediately understand the purpose: stabilizing a flaky CI job by excluding a problematic external link.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

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

@codecov
Copy link

codecov bot commented Oct 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 50.20%. Comparing base (5d0e9b0) to head (a628337).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #35   +/-   ##
=======================================
  Coverage   50.20%   50.20%           
=======================================
  Files          13       13           
  Lines         249      249           
  Branches       26       26           
=======================================
  Hits          125      125           
  Misses        119      119           
  Partials        5        5           

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5d0e9b0 and a628337.

📒 Files selected for processing (1)
  • .github/workflows/build.yml (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (5)
  • GitHub Check: Audit - Production
  • GitHub Check: Audit - All
  • GitHub Check: Lint & Format
  • GitHub Check: build
  • GitHub Check: check_release
🔇 Additional comments (1)
.github/workflows/build.yml (1)

189-189: The original review comment is incorrect. Only one ignore_links occurrence exists in the workflow file (line 189). There is no second location to update, so no inconsistency with the change exists.

Likely an incorrect or invalid review comment.

@andyjakubowski andyjakubowski merged commit 90cbb23 into main Oct 17, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants