Skip to content

Add workflow to comment on PRs about integration tests - #55

Merged
shotarok merged 1 commit into
mainfrom
add-integration-test-comment-workflow
Feb 17, 2026
Merged

Add workflow to comment on PRs about integration tests#55
shotarok merged 1 commit into
mainfrom
add-integration-test-comment-workflow

Conversation

@shotarok

Copy link
Copy Markdown
Owner

Summary

This PR adds a new GitHub Actions workflow that automatically comments on newly created PRs with instructions on how to trigger the integration tests required for merge.

Changes

  • Added .github/workflows/add-integration-test-comment.yml

How it works

The new workflow:

  1. Triggers when a PR is opened or reopened
  2. Adds a helpful comment explaining that contributors need to add the ready-to-merge label to trigger integration tests
  3. Describes what the integration tests include (Ansible syntax check, playbook execution, idempotence verification)
  4. Notes that if tests fail, the label is automatically removed

This integrates with the existing ansible-playbook.yml workflow which runs integration tests when the ready-to-merge label is present.

@shotarok can click here to continue refining the PR

This workflow automatically adds a helpful comment to new PRs explaining
how to trigger the integration tests required for merge. Contributors
need to add the 'ready-to-merge' label to trigger the integration tests.
@github-actions

Copy link
Copy Markdown

🚀 Integration Test Required for Merge

Hi! 👋 Thanks for opening this pull request.

To merge this PR, you need to trigger the integration tests first. Here's how:

Trigger Integration Tests

Once your changes are ready for testing, add the ready-to-merge label to this PR. This will automatically trigger the integration test workflow.

You can add the label by:

  1. Going to this PR's page on GitHub
  2. Click on the "Labels" dropdown (right sidebar)
  3. Select "ready-to-merge"

What happens next?

  • The integration tests will run on macOS
  • Tests include: Ansible syntax check, playbook execution, and idempotence verification
  • If all tests pass, your PR will be ready for merge! 🎉

Note: If tests fail, the ready-to-merge label will be automatically removed. You'll need to fix the issues and add the label again to re-run the tests.


This comment is automatically added by the repository's CI/CD workflow.

@shotarok shotarok added the ready-to-merge PR is ready to be merged label Feb 17, 2026
@shotarok
shotarok enabled auto-merge (rebase) February 17, 2026 02:57
@shotarok
shotarok merged commit eb4761f into main Feb 17, 2026
3 checks passed
@shotarok
shotarok deleted the add-integration-test-comment-workflow branch February 17, 2026 03:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge PR is ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants