Skip to content

test: trigger integ-test-auto workflow (DO NOT MERGE)#8

Open
ozelalisen wants to merge 3 commits intomainfrom
test/trigger-integ-auto
Open

test: trigger integ-test-auto workflow (DO NOT MERGE)#8
ozelalisen wants to merge 3 commits intomainfrom
test/trigger-integ-auto

Conversation

@ozelalisen
Copy link
Copy Markdown
Owner

Dummy snapshot change to test the two-stage workflow_run pattern for auto integ tests.

The pull_request_review trigger has no access to repository secrets
for fork PRs, breaking team membership checks and OIDC auth.

Split into two-stage workflow_run pattern (matching pr-linter):
- Stage 1 (trigger): Unprivileged pull_request_review saves PR
  metadata and snapshot check result as artifacts
- Stage 2 (deployment): Privileged workflow_run downloads artifacts,
  validates team membership with secrets, runs integ tests
Temporary commit to validate the workflow_run artifact handoff on
ozelalisen/aws-cdk fork. Will be reverted after testing.
Copy link
Copy Markdown

@amazon-q-developer amazon-q-developer bot left a comment

Choose a reason for hiding this comment

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

This PR is marked "DO NOT MERGE" and is intended for testing the two-stage workflow_run pattern for automatic integration tests. The changes consist of documentation describing the integration test deployment workflow improvements and generated artifacts (compiled JavaScript files, type definitions, and test snapshots).

Since this is a test PR not intended for production merge, no blocking issues have been identified. The workflow testing objective appears clear from the documentation and snapshot changes included.


You can now have the agent implement changes and create commits directly on your pull request's source branch. Simply comment with /q followed by your request in natural language to ask the agent to make changes.


⚠️ This PR contains more than 30 files. Amazon Q is better at reviewing smaller PRs, and may miss issues in larger changesets.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 23, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ☑️SkippedFailed ❌️
Security Guardian Results24 ran23 passed1 failed
TestResult
Security Guardian Results
packages/@aws-cdk/aws-pipes-enrichments-alpha/test/integ.api-destination.js.snapshot/aws-cdk-pipes-enrichments-lambda.template.json
sqs-encryption-enabled.guard❌ failure

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 23, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ☑️SkippedFailed ❌️
Security Guardian Results with resolved templates24 ran23 passed1 failed
TestResult
Security Guardian Results with resolved templates
packages/@aws-cdk/aws-pipes-enrichments-alpha/test/integ.api-destination.js.snapshot/aws-cdk-pipes-enrichments-lambda.template.json
sqs-encryption-enabled.guard❌ failure

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant