search

astronomer / astro-provider-ray

This provider contains operators, decorators and triggers to send a ray job from an airflow task
https://astronomer.github.io/astro-provider-ray/
Apache License 2.0
12 stars 2 forks source link

Bug fix: Allowing tests to run for PRs from forked repos #72

Closed venkatajagannath closed 1 month ago

venkatajagannath commented 1 month ago

The test workflow runs integration tests using an external GKE cluster and therefore requires some credentials.

The integration test currently fails when it is run on PRs from forked repos.

This PR fixes this issue using pull_request_target.

See here for more information: https://github.com/astronomer/astronomer-cosmos/blob/main/.github/workflows/test.yml https://datachain.ai/blog/testing-external-contributions-using-github-actions-secrets

codecov-commenter commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.42%. Comparing base (a783526) to head (a3afe2d). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #72 +/- ## ======================================= Coverage 95.42% 95.42% ======================================= Files 5 5 Lines 546 546 ======================================= Hits 521 521 Misses 25 25 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.