Add github actions sync resolver #539
Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #539 +/- ##
========================================
+ Coverage 8.13% 8.19% +0.05%
========================================
Files 278 279 +1
Lines 45874 45915 +41
========================================
+ Hits 3732 3762 +30
- Misses 41866 41878 +12
+ Partials 276 275 -1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
cf752bb to
c2a6810
Compare
c2a6810 to
b7a1e66
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR adds GitHub Actions support to the sync reporter by implementing a GitHub Actions-specific environment resolver. The changes enable automatic detection of GitHub Actions execution context and extraction of relevant metadata like repository URL, git references, and trigger types.
- Introduces a new GitHub Actions environment resolver that reads GitHub-specific environment variables
- Updates the sync reporter to automatically detect and use GitHub Actions context when available
- Adds comprehensive test coverage for the new environment resolver functionality
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| scan.go | Updates sync reporter initialization to use the new environment resolver factory function |
| pkg/reporter/sync.go | Adds factory function to automatically select appropriate environment resolver based on context |
| pkg/reporter/github_sync_resolver.go | Implements GitHub Actions-specific environment resolver with mapping of GitHub events to tool triggers |
| pkg/reporter/sync_test.go | Adds comprehensive test coverage for the new environment resolver functionality |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Sahil Bansal <bansalsahil315@gmail.com>
|
@Sahilb315 The tests are failing for this. |
Ohh.. they are passing locally but failing here because the env variables are set in the CI/CD runner & can access the variables. @abhisek Fixed it |
Fixes #530