Skip to content

Remove ready_for_review trigger from run-tests workflow#56

Merged
albertodebortoli merged 1 commit intomainfrom
remove-ready-for-review-trigger
Apr 8, 2026
Merged

Remove ready_for_review trigger from run-tests workflow#56
albertodebortoli merged 1 commit intomainfrom
remove-ready-for-review-trigger

Conversation

@albertodebortoli
Copy link
Copy Markdown
Member

Pull Request Title

Remove ready_for_review trigger from run-tests workflow

Description

  • Removes the ready_for_review event type from the pull_request trigger in the run-tests.yml workflow.
  • Tests will no longer be re-triggered when a draft PR is marked as ready for review; they already run on opened, reopened, and synchronize.

Type of Change

  • Feature
  • Bug fix
  • Maintenance / Refactor
  • Documentation
  • CI / Tooling
  • Other (specify)

How Has This Been Tested?

  • Manually tested locally (describe)
  • Verified workflow YAML is valid after the change.

Checklist

  • Swift code builds locally (swift build)
  • Tests pass locally (swift test)
  • Code style / formatting respected
  • Documentation updated (README / comments)
  • Version / tag alignment considered (if release related)
  • PR title follows conventional style (optional)

CI Considerations

  • Affects build time notably
  • Requires new secrets / env vars
  • Alters release process

Breaking Changes?

  • No
  • Yes (describe impact and migration path)

Additional Notes

N/A

@albertodebortoli albertodebortoli force-pushed the remove-ready-for-review-trigger branch from 8b62a77 to 1b7e9db Compare April 8, 2026 22:54
@albertodebortoli albertodebortoli added the maintenance Maintenance work label Apr 8, 2026
@albertodebortoli albertodebortoli added this to the 0.14.0 milestone Apr 8, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@albertodebortoli albertodebortoli merged commit d1eac1d into main Apr 8, 2026
3 checks passed
@albertodebortoli albertodebortoli deleted the remove-ready-for-review-trigger branch April 8, 2026 22:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Maintenance work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant