Skip to content

Run Lint CI for docs change only#109

Merged
TxCorpi0x merged 3 commits into
masterfrom
mehdi/ci-doc-only-run
Mar 24, 2026
Merged

Run Lint CI for docs change only#109
TxCorpi0x merged 3 commits into
masterfrom
mehdi/ci-doc-only-run

Conversation

@TxCorpi0x

@TxCorpi0x TxCorpi0x commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

Description

This pull request updates the CI workflow to optimize job execution based on detected changes in the repository. The main improvement is the introduction of a change detection job that distinguishes between documentation and code changes, allowing subsequent CI steps to conditionally run only when relevant files are modified.

Enhancements to CI workflow efficiency:

  • Added a detect-changes job to .github/workflows/ci.yml that uses dorny/paths-filter@v2 to identify whether docs or code files were changed, exposing these results as outputs for downstream jobs.
  • Updated the ci job to depend on detect-changes and conditionally run CI steps based on the detected changes: steps now execute when manually triggered, when code changes are detected, or when docs changes are detected (lint step only).

Reviewers checklist:

  • Try to write more meaningful comments with clear actions to be taken.
  • Nit-picking should be unblocking. Focus on core issues.

Authors checklist

  • Provide a concise and meaningful description
  • Review the code yourself first, before making the PR.
  • Annotate your PR in places that require explanation.
  • Think and try to split the PR to smaller PR if it is big.

This change is Reviewable

@TxCorpi0x TxCorpi0x marked this pull request as ready for review March 18, 2026 09:37
@TxCorpi0x TxCorpi0x requested a review from a team as a code owner March 18, 2026 09:37
@TxCorpi0x TxCorpi0x requested review from a team, akhlopiachyi, masihyeganeh, metalarm10, miladz68 and ysv and removed request for a team March 18, 2026 09:37

@ysv ysv left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@ysv reviewed 1 file and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on akhlopiachyi, masihyeganeh, metalarm10, and miladz68).

@TxCorpi0x TxCorpi0x merged commit f6e155e into master Mar 24, 2026
14 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