Skip to content

fix release ci#1770

Merged
javeme merged 2 commits intomasterfrom
fix_release
Mar 21, 2022
Merged

fix release ci#1770
javeme merged 2 commits intomasterfrom
fix_release

Conversation

@coderzc
Copy link
Copy Markdown
Member

@coderzc coderzc commented Mar 2, 2022

fix #1766

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 2, 2022

Codecov Report

Merging #1770 (572de9b) into master (5cdf770) will increase coverage by 7.57%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1770      +/-   ##
============================================
+ Coverage     63.26%   70.84%   +7.57%     
- Complexity      684      976     +292     
============================================
  Files           443      446       +3     
  Lines         37738    37802      +64     
  Branches       5381     5392      +11     
============================================
+ Hits          23875    26780    +2905     
+ Misses        11247     8295    -2952     
- Partials       2616     2727     +111     
Impacted Files Coverage Δ
...n/java/com/baidu/hugegraph/version/ApiVersion.java 75.00% <0.00%> (ø)
...ugegraph/api/traversers/ResourceAllocationAPI.java 90.90% <0.00%> (ø)
.../baidu/hugegraph/api/traversers/AdamicAdarAPI.java 91.66% <0.00%> (ø)
...graph/traversal/algorithm/PredictionTraverser.java 86.66% <0.00%> (ø)
...du/hugegraph/traversal/optimize/TraversalUtil.java 62.60% <0.00%> (+0.21%) ⬆️
.../com/baidu/hugegraph/auth/StandardAuthManager.java 92.81% <0.00%> (+0.32%) ⬆️
...idu/hugegraph/schema/builder/EdgeLabelBuilder.java 83.62% <0.00%> (+0.34%) ⬆️
.../java/com/baidu/hugegraph/backend/query/Query.java 86.11% <0.00%> (+0.39%) ⬆️
...hugegraph/backend/query/ConditionQueryFlatten.java 75.83% <0.00%> (+0.41%) ⬆️
...n/java/com/baidu/hugegraph/structure/HugeEdge.java 76.23% <0.00%> (+0.44%) ⬆️
... and 127 more

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 5cdf770...572de9b. Read the comment docs.

mvn clean package -DskipTests
mv "$(find hugegraph-*.tar.gz | head -1)" hugegraph-${TAG_NAME/#v/}.tar.gz

- name: Rename file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

why need to split into 2 stages? can put "mvn clean package -DskipTests" into "Rename file" stage?

Copy link
Copy Markdown
Member Author

@coderzc coderzc Mar 7, 2022

Choose a reason for hiding this comment

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

because of the "Rename file" operation is not unessential

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

OK

mv "$(find hugegraph-*.tar.gz | head -1)" hugegraph-${TAG_NAME/#v/}.tar.gz

- name: Rename file
if: contains(env.TAG_NAME, "-")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

fixed error mv: '***-0.12.0.tar.gz' and '***-0.12.0.tar.gz' are the same file

if: contains(env.TAG_NAME, "-")
run: mv "$(find hugegraph-*.tar.gz | head -1)" hugegraph-${TAG_NAME/#v/}.tar.gz

- uses: xresloader/upload-to-github-release@master
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also add name "Upload to GitHub"

mvn clean package -DskipTests
mv "$(find hugegraph-*.tar.gz | head -1)" hugegraph-${TAG_NAME/#v/}.tar.gz

- name: Rename file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

OK

mv "$(find hugegraph-*.tar.gz | head -1)" hugegraph-${TAG_NAME/#v/}.tar.gz

- name: Rename file
if: contains(env.TAG_NAME, "-")
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

only need to rename when TAG_NAME is like 0.12.0-b2?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yes

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 18, 2022

CLA Assistant Lite bot Good! All Contributors have signed the CLA.

Copy link
Copy Markdown
Member

@imbajin imbajin left a comment

Choose a reason for hiding this comment

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

could sign new CLA again

@coderzc
Copy link
Copy Markdown
Member Author

coderzc commented Mar 21, 2022

I have read the CLA Document and I hereby sign the CLA

github-actions bot added a commit that referenced this pull request Mar 21, 2022
@javeme javeme merged commit 14eb6cb into master Mar 21, 2022
@javeme javeme deleted the fix_release branch March 21, 2022 05:54
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.

[Feature] maven仓库还未收录0.12.0版本

3 participants