Skip to content

Remove 'if' condition from check-links job#1018

Merged
egorikftp merged 1 commit into
mainfrom
egorikftp-patch-1
Jun 10, 2026
Merged

Remove 'if' condition from check-links job#1018
egorikftp merged 1 commit into
mainfrom
egorikftp-patch-1

Conversation

@egorikftp

Copy link
Copy Markdown
Member

Remove conditional check for check-links job.


📝 Changelog

If this PR introduces user-facing changes, please update the relevant Unreleased section in changelogs:

Remove conditional check for check-links job.
@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: cd7bedbe-785f-4867-be70-643519979fb2

📥 Commits

Reviewing files that changed from the base of the PR and between 1189fb0 and e6aa834.

📒 Files selected for processing (1)
  • .github/workflows/validation.yml
💤 Files with no reviewable changes (1)
  • .github/workflows/validation.yml

Walkthrough

This PR removes the if: false guard from the check-links job in the validation workflow, enabling link validation checks to execute as part of the standard validation pipeline.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: removing the 'if' condition from the check-links job in the workflow file.
Description check ✅ Passed The description includes the changelog template with all required sections, though no changelog updates were marked as necessary since this is an internal workflow change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch egorikftp-patch-1

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@egorikftp egorikftp merged commit adc9883 into main Jun 10, 2026
4 checks passed
@egorikftp egorikftp deleted the egorikftp-patch-1 branch June 10, 2026 10:21
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.

1 participant