Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate to official Gradle GitHub action. #3967

Merged
merged 1 commit into from
Dec 9, 2021

Conversation

anuraaga
Copy link
Contributor

@anuraaga anuraaga commented Dec 8, 2021

No description provided.

@codecov
Copy link

codecov bot commented Dec 8, 2021

Codecov Report

Merging #3967 (b823065) into main (76ae8d2) will increase coverage by 0.06%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main    #3967      +/-   ##
============================================
+ Coverage     89.84%   89.90%   +0.06%     
- Complexity     4205     4208       +3     
============================================
  Files           503      503              
  Lines         12752    12752              
  Branches       1232     1232              
============================================
+ Hits          11457    11465       +8     
+ Misses          902      896       -6     
+ Partials        393      391       -2     
Impacted Files Coverage Δ
...telemetry/sdk/trace/export/BatchSpanProcessor.java 87.76% <0.00%> (-0.72%) ⬇️
...ntelemetry/sdk/extension/resources/OsResource.java 90.69% <0.00%> (+4.65%) ⬆️
...ava/io/opentelemetry/sdk/internal/RateLimiter.java 100.00% <0.00%> (+5.88%) ⬆️
...metry/sdk/extension/resources/ProcessResource.java 87.50% <0.00%> (+6.25%) ⬆️
...elemetry/sdk/extension/resources/HostResource.java 92.30% <0.00%> (+15.38%) ⬆️
...dk/extension/resources/ProcessRuntimeResource.java 100.00% <0.00%> (+16.66%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76ae8d2...b823065. Read the comment docs.

@iNikem
Copy link
Contributor

iNikem commented Dec 8, 2021

Out of curiosity: why this change?

@anuraaga
Copy link
Contributor Author

anuraaga commented Dec 8, 2021

@iNikem Noticed the official action, since we disable the remote cache support of the old one anyways, this seems to have similar features but more active maintenance.

@anuraaga anuraaga merged commit 3f05d20 into open-telemetry:main Dec 9, 2021
This was referenced Dec 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants