Skip to content

ci: limit ClusterFuzzLite workflow to src/ and .clusterfuzzlite/ changes#2304

Merged
manishdait merged 4 commits into
hiero-ledger:mainfrom
sudo-anshul:ci/limit-clusterfuzzlite-paths
May 22, 2026
Merged

ci: limit ClusterFuzzLite workflow to src/ and .clusterfuzzlite/ changes#2304
manishdait merged 4 commits into
hiero-ledger:mainfrom
sudo-anshul:ci/limit-clusterfuzzlite-paths

Conversation

@sudo-anshul
Copy link
Copy Markdown
Contributor

Description

add paths: filters to the ClusterFuzzLite workflow triggers so it only runs
when files in src/ or .clusterfuzzlite/ are modified. this follows the
established pattern used by other workflows (e.g., pr-check-primary-codecov.yml).

fixes #2301

Signed-off-by: Anshul Kushwaha <137977998+sudo-anshul@users.noreply.github.com>
@sudo-anshul sudo-anshul requested review from a team as code owners May 22, 2026 12:44
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 22, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: ba509e23-2aef-4f18-94b7-386bf6f47af1

📥 Commits

Reviewing files that changed from the base of the PR and between 91b6504 and e352715.

📒 Files selected for processing (1)
  • .github/workflows/clusterfuzzlite.yml

Walkthrough

The ClusterFuzzLite GitHub Actions workflow now includes path filters on both pull_request and push triggers so it runs only when src/**, .clusterfuzzlite/**, pyproject.toml, or generate_proto.py change; push remains limited to main.

Changes

ClusterFuzzLite Workflow Path Filtering

Layer / File(s) Summary
ClusterFuzzLite trigger path filters
.github/workflows/clusterfuzzlite.yml
Both pull_request and push workflow triggers now include paths filters for src/**, .clusterfuzzlite/**, pyproject.toml, and generate_proto.py, with push still constrained to main.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately summarizes the main change: limiting the ClusterFuzzLite workflow to only run when src/ and .clusterfuzzlite/ directories are modified.
Description check ✅ Passed The description is directly related to the changeset, explaining the purpose of adding path filters to the ClusterFuzzLite workflow and referencing the linked issue #2301.
Linked Issues check ✅ Passed The pull request meets all coding requirements from issue #2301: path filters were added to both pull_request and push triggers for src/** and .clusterfuzzlite/**, following the established pattern.
Out of Scope Changes check ✅ Passed All changes are within scope, limited to updating the .github/workflows/clusterfuzzlite.yml file with path filters as specified in issue #2301, with no unrelated modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

📋 Issue Planner

Built with CodeRabbit's Coding Plans for faster development and fewer bugs.

View plan used: #2301

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

Copy link
Copy Markdown
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 info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: ce7714c3-0441-4f9a-8656-d668ea5005c2

📥 Commits

Reviewing files that changed from the base of the PR and between 08784be and 531b810.

📒 Files selected for processing (1)
  • .github/workflows/clusterfuzzlite.yml

Comment thread .github/workflows/clusterfuzzlite.yml
@github-actions github-actions Bot added open to community review PR is open for community review and feedback queue:junior-committer PR awaiting initial quality review labels May 22, 2026
Signed-off-by: Anshul Kushwaha <137977998+sudo-anshul@users.noreply.github.com>
@github-actions github-actions Bot added Good First Issue Issues which are ideal for a first time or new project contributor. scope: CI/CD involves continuous integration or delivery labels May 22, 2026
@manishdait manishdait added this to the v0.2.7 milestone May 22, 2026
@github-actions github-actions Bot added queue:committers PR awaiting committer technical review and removed queue:junior-committer PR awaiting initial quality review labels May 22, 2026
@aceppaluni aceppaluni added the status: update branch developer needs to click update branch label May 22, 2026
@exploreriii exploreriii added step: merge ready PR is ready and waiting to merge and removed queue:committers PR awaiting committer technical review open to community review PR is open for community review and feedback labels May 22, 2026
@github-actions github-actions Bot added open to community review PR is open for community review and feedback status: ready-to-merge PR has 1+ maintainer and 2+ total approvals, ready to merge labels May 22, 2026
@manishdait manishdait merged commit 37f4bee into hiero-ledger:main May 22, 2026
10 checks passed
@github-actions
Copy link
Copy Markdown

👋 Hi @sudo-anshul! Great work completing a Good First Issue issue! 🎉

Thanks for your contribution! 🚀

Here are some issues you might want to explore next:

🌟 Stay connected:

Happy coding! 🚀
— Hiero Python SDK Team

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

Labels

Good First Issue Issues which are ideal for a first time or new project contributor. open to community review PR is open for community review and feedback scope: CI/CD involves continuous integration or delivery status: ready-to-merge PR has 1+ maintainer and 2+ total approvals, ready to merge status: update branch developer needs to click update branch step: merge ready PR is ready and waiting to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Limit ClusterFuzzLite workflow execution to changes in src/ and .clusterfuzzlite/

4 participants