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

[Issue-491] Wrong count number of the artifact #492

Conversation

thekingofcity
Copy link
Contributor

Signed-off-by: zhongle.wang zhongle_wang@dell.com

Change log description

A typical name of the artiface should be something like 0.10.0-1.ff9f6d1-SNAPSHOT. With the [version]-[commit count]-[commit sha]-SNAPSHOT format. But for artifacts produced by Github Actions, the commit count are always set to 1. This is caused by the depth==1 clone, so we change it to unlimited.

Purpose of the change

Fixes #491

What the code does

Add a parameter to specify the clone depth to be unlimited.

How to verify it

Artifacts build by the new yaml should have the corret commit count.

See the List of artifacts section of this run.

Signed-off-by: zhongle.wang <zhongle_wang@dell.com>
@codecov
Copy link

codecov bot commented Apr 16, 2021

Codecov Report

Merging #492 (61c8e34) into master (b3eb48e) will increase coverage by 0.25%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #492      +/-   ##
============================================
+ Coverage     83.81%   84.06%   +0.25%     
- Complexity      422      424       +2     
============================================
  Files            35       35              
  Lines          1946     1946              
  Branches        206      206              
============================================
+ Hits           1631     1636       +5     
+ Misses          170      167       -3     
+ Partials        145      143       -2     
Impacted Files Coverage Δ Complexity Δ
...ega/connectors/flink/FlinkPravegaOutputFormat.java 83.95% <0.00%> (+6.17%) 20.00% <0.00%> (+2.00%)

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 b3eb48e...61c8e34. Read the comment docs.

Copy link
Contributor

@crazyzhou crazyzhou left a comment

Choose a reason for hiding this comment

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

LGTM

@crazyzhou crazyzhou merged commit 7a8ae23 into pravega:master Apr 16, 2021
@thekingofcity thekingofcity deleted the issue-491-Wrong-count-number-of-the-artifact branch April 20, 2021 06:44
crazyzhou pushed a commit that referenced this pull request Apr 22, 2021
Signed-off-by: zhongle.wang <zhongle_wang@dell.com>
crazyzhou pushed a commit that referenced this pull request Apr 22, 2021
Signed-off-by: zhongle.wang <zhongle_wang@dell.com>
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.

Wrong count number of the artifact
2 participants