Skip to content

Conversation

@JBAhire
Copy link
Member

@JBAhire JBAhire commented Jan 6, 2021

Description

This PR uses new custom actions for

  • Gradle
  • Validate charts and publish

changes as per discussions here: hypertrace/query-service#47

Testing

tested changes here: #65

Checklist:

  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • Any dependent changes have been merged and published in downstream modules

Documentation

@codecov
Copy link

codecov bot commented Jan 6, 2021

Codecov Report

Merging #66 (9b912ca) into main (3f8924f) will decrease coverage by 12.94%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##               main      #66       +/-   ##
=============================================
- Coverage     80.71%   67.77%   -12.95%     
+ Complexity      222      203       -19     
=============================================
  Files            27       24        -3     
  Lines           752      692       -60     
  Branches         56       56               
=============================================
- Hits            607      469      -138     
- Misses           98      187       +89     
+ Partials         47       36       -11     
Flag Coverage Δ Complexity Δ
integration ? ?
unit 67.77% <ø> (ø) 0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ Complexity Δ
...tribute/service/model/AttributeMetadataDocKey.java 0.00% <0.00%> (-100.00%) 0.00% <0.00%> (ø%)
...e/core/attribute/service/AttributeServiceImpl.java 27.72% <0.00%> (-45.05%) 11.00% <0.00%> (ø%)
.../core/attribute/service/AttributeServiceEntry.java
...tribute/service/client/AttributeServiceClient.java
...ce/client/config/AttributeServiceClientConfig.java

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 3f8924f...9b912ca. Read the comment docs.

@github-actions

This comment has been minimized.

@JBAhire JBAhire merged commit 59e0854 into main Jan 6, 2021
@JBAhire JBAhire deleted the uses-latest-actions branch January 6, 2021 04:37
@github-actions
Copy link
Contributor

github-actions bot commented Jan 6, 2021

Unit Test Results

20 files  20 suites   9s ⏱️
75 tests 75 ✔️ 0 💤 0 ❌

Results for commit 59e0854.

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