Skip to content

[to #526] Add codecov on Github Action (#524)#533

Merged
zz-jason merged 2 commits into
tikv:release-3.1from
ti-srebot:release-3.1-49476d7fa0db
Feb 21, 2022
Merged

[to #526] Add codecov on Github Action (#524)#533
zz-jason merged 2 commits into
tikv:release-3.1from
ti-srebot:release-3.1-49476d7fa0db

Conversation

@ti-srebot

@ti-srebot ti-srebot commented Feb 18, 2022

Copy link
Copy Markdown
Collaborator

cherry-pick #524 to release-3.1
You can switch your code base to this Pull Request by using git-extras:

# In client-java repo:
git pr https://github.com/tikv/client-java/pull/533

After apply modifications, you can push your change to this PR via:

git push git@github.com:ti-srebot/client-java.git pr/533:release-3.1-49476d7fa0db

Signed-off-by: andylokandy andylokandy@hotmail.com

Ref #526

What is changed and how it works?

Add CI using Github Action. Intended to replace jenkins.

Code changes

  • No code

Check List for Tests

  • No code

Signed-off-by: ti-srebot <ti-srebot@pingcap.com>
@ti-srebot

Copy link
Copy Markdown
Collaborator Author

/run-all-tests

@codecov

codecov Bot commented Feb 18, 2022

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (release-3.1@3cde9f0). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff               @@
##             release-3.1     #533   +/-   ##
==============================================
  Coverage               ?   28.28%           
  Complexity             ?     1088           
==============================================
  Files                  ?      263           
  Lines                  ?    16525           
  Branches               ?     1892           
==============================================
  Hits                   ?     4674           
  Misses                 ?    11330           
  Partials               ?      521           

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 3cde9f0...419d732. Read the comment docs.

@zz-jason zz-jason left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@andylokandy please resolve conflicts.

Signed-off-by: andylokandy <andylokandy@hotmail.com>
@andylokandy

Copy link
Copy Markdown
Contributor

@marsishandsome @zz-jason PTAL

@zz-jason zz-jason left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@marsishandsome marsishandsome left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@andylokandy

Copy link
Copy Markdown
Contributor

/merge

@ti-srebot

Copy link
Copy Markdown
Collaborator Author

/run-all-tests

@andylokandy

Copy link
Copy Markdown
Contributor

@marsishandsome We have to forcibly merge it.

@zz-jason zz-jason merged commit bbf9e19 into tikv:release-3.1 Feb 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants