Skip to content

Add external contributor unit tests workflow#6814

Closed
RanVaknin wants to merge 1 commit intomasterfrom
rvaknin/add-external-contributor-unit-test-workflow
Closed

Add external contributor unit tests workflow#6814
RanVaknin wants to merge 1 commit intomasterfrom
rvaknin/add-external-contributor-unit-test-workflow

Conversation

@RanVaknin
Copy link
Copy Markdown
Contributor

Context

Adding a "pure" Github workflow to allow external contributors to run our CI unit tests.
With this change external contributors can fork our repo, and run the same CI test suite using infrastructure provided by the Github workflow feature. This eliminates the need for authentication, as the workflow is read only.

In their fork, external contributors will be able to visit the actions tab, and run this new workflow on their desired branch:

image

Testing

Tested on my private fork: https://github.com/RanVaknin/aws-sdk-java-v2/actions/runs/23517845706

@RanVaknin RanVaknin requested a review from a team as a code owner March 24, 2026 23:58
@RanVaknin RanVaknin added changelog-not-required Indicate changelog entry is not required for a specific PR no-api-surface-area-change Indicate there is no API surface area change and thus API surface area review is not required labels Mar 24, 2026
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@RanVaknin RanVaknin closed this Mar 25, 2026
@github-actions
Copy link
Copy Markdown

This pull request has been closed and the conversation has been locked. Comments on closed PRs are hard for our team to see. If you need more assistance, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

changelog-not-required Indicate changelog entry is not required for a specific PR no-api-surface-area-change Indicate there is no API surface area change and thus API surface area review is not required

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant