Skip to content

fix release ci - #1770

Merged
javeme merged 2 commits into
masterfrom
fix_release
Mar 21, 2022
Merged

fix release ci#1770
javeme merged 2 commits into
masterfrom
fix_release

Conversation

@coderzc

@coderzc coderzc commented Mar 2, 2022

Copy link
Copy Markdown
Member

fix #1766

@codecov

codecov Bot commented Mar 2, 2022

Copy link
Copy Markdown

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?

@coderzc coderzc Mar 7, 2022

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.

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

Comment thread .github/workflows/release.yml Outdated
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

github-actions Bot commented Mar 18, 2022

Copy link
Copy Markdown

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

@imbajin imbajin 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.

could sign new CLA again

@coderzc

coderzc commented Mar 21, 2022

Copy link
Copy Markdown
Member Author

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