Skip to content

chore: remove reward workflow#354

Merged
i582 merged 1 commit into
developfrom
danil42russia/remove_reward
Jun 6, 2026
Merged

chore: remove reward workflow#354
i582 merged 1 commit into
developfrom
danil42russia/remove_reward

Conversation

@Danil42Russia
Copy link
Copy Markdown
Contributor

@Danil42Russia Danil42Russia commented Jun 2, 2026

Issue

Closed due to the closure of Ton Society

Checklist

Please ensure the following items are completed before requesting review:

  • Updated CHANGELOG.md with relevant changes
  • Documented the contribution in README.md
  • Added tests to demonstrate correct behavior (both positive and negative cases)
  • All tests pass successfully (yarn test)
  • Code passes linting checks (yarn lint)

Summary by CodeRabbit

  • Chores
    • Removed automated reward workflows previously triggered on merged pull requests and manual dispatch.
    • Updated QA workflow configuration to adjust trigger rules for workflow file changes.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 2, 2026

Review Change Stack

📝 Walkthrough

Walkthrough

The PR removes two reward contribution workflows (reward-merged-prs.yml and reward.yml) and updates the QA workflow to stop ignoring changes to those files. This cleanup ensures that QA checks will be triggered if similar workflows are added again in the future.

Changes

Reward Workflows Cleanup

Layer / File(s) Summary
QA workflow path-ignore filters
.github/workflows/qa.yaml
Removed .github/workflows/reward*.yml from paths-ignore in both push and pull_request events, allowing future modifications to reward workflows to trigger QA checks.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 The reward scripts fade away so fast,
QA's watchful eye holds steadfast,
No more patterns whisper "ignore,"
Workflows dance from floor to floor,
Cleanup magic, clean and bright! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The PR description is incomplete: it lacks an issue number, CHANGELOG.md update, and clear rationale for removal despite checking some checklist items. Add the issue number (e.g., 'Closes #XXX'), update CHANGELOG.md as indicated in the checklist, and provide a brief explanation of why the reward workflows are being removed.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: remove reward' is clear and directly matches the changeset, which removes reward workflow files and updates workflow filters.
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 docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch danil42russia/remove_reward

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.

@i582 i582 changed the title chore: remove reward chore: remove reward workflow Jun 6, 2026
@i582 i582 merged commit 64e8aa1 into develop Jun 6, 2026
5 checks passed
@i582 i582 deleted the danil42russia/remove_reward branch June 6, 2026 23:30
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.

2 participants