openshift-pipelines / pipelines-as-code

Pipelines-as-Code for Tekton
https://pipelinesascode.com
Apache License 2.0
124 stars 81 forks source link

added support for azure devops #1654

Open aayeshaarshad opened 2 months ago

aayeshaarshad commented 2 months ago
chmouel commented 2 months ago

/ok-to-test

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 36.24535% with 343 lines in your changes are missing coverage. Please review.

Project coverage is 62.97%. Comparing base (5ecb9ea) to head (a1af7e8). Report is 53 commits behind head on main.

:exclamation: Current head a1af7e8 differs from pull request most recent head fd7c2c3. Consider uploading reports for the commit fd7c2c3 to get more accurate results

Files Patch % Lines
pkg/provider/azuredevops/azuredevops.go 21.91% 279 Missing and 6 partials :warning:
pkg/provider/azuredevops/acl.go 0.00% 19 Missing :warning:
pkg/provider/azuredevops/parse_payload.go 85.14% 10 Missing and 5 partials :warning:
pkg/reconciler/event.go 0.00% 11 Missing and 1 partial :warning:
pkg/kubeinteraction/labels.go 25.00% 4 Missing and 2 partials :warning:
pkg/adapter/adapter.go 40.00% 2 Missing and 1 partial :warning:
pkg/provider/azuredevops/detect.go 88.88% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1654 +/- ## ========================================== - Coverage 64.31% 62.97% -1.34% ========================================== Files 140 144 +4 Lines 10789 11326 +537 ========================================== + Hits 6939 7133 +194 - Misses 3334 3661 +327 - Partials 516 532 +16 ```

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

chmouel commented 2 months ago

/test go-testing

aayeshaarshad commented 1 month ago

@chmouel any update on review?

rasheedamir commented 1 month ago

@chmouel we will appreciate if you can please review it; as we would like this be merged into main branch! can you confirm when you can review it?

we have two OpenShift customers and who really wants to use this and they want it to be part of the main release

rasheedamir commented 3 weeks ago

@chmouel an reminder; any updates on this?

rasheedamir commented 3 days ago

@chmouel any updates on this? we are really surprised how poor the response time is! I will appreciate if we can hear back