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

fix(dist): bin package lack execute permission #394

Merged
merged 3 commits into from
Dec 9, 2022

Conversation

z7658329
Copy link
Member

@z7658329 z7658329 commented Dec 8, 2022

No description provided.

@codecov
Copy link

codecov bot commented Dec 8, 2022

Codecov Report

Merging #394 (adca83a) into master (942e52c) will increase coverage by 0.03%.
The diff coverage is n/a.

❗ Current head adca83a differs from pull request most recent head 2b0620a. Consider uploading reports for the commit 2b0620a to get more accurate results

@@             Coverage Diff              @@
##             master     #394      +/-   ##
============================================
+ Coverage     62.55%   62.58%   +0.03%     
+ Complexity     1866      894     -972     
============================================
  Files           260       91     -169     
  Lines          9405     4383    -5022     
  Branches        872      515     -357     
============================================
- Hits           5883     2743    -3140     
+ Misses         3140     1437    -1703     
+ Partials        382      203     -179     
Impacted Files Coverage Δ
...rc/main/java/org/apache/hugegraph/util/IdUtil.java
.../hugegraph/structure/traverser/RepeatEdgeStep.java
.../hugegraph/exception/InvalidResponseException.java
...a/org/apache/hugegraph/api/traverser/EdgesAPI.java
...g/apache/hugegraph/structure/schema/EdgeLabel.java
.../main/java/org/apache/hugegraph/util/JsonUtil.java
...ava/org/apache/hugegraph/api/graphs/GraphsAPI.java
...ache/hugegraph/api/traverser/TemplatePathsAPI.java
...in/java/org/apache/hugegraph/api/auth/AuthAPI.java
...java/org/apache/hugegraph/api/graph/VertexAPI.java
... and 159 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

hugegraph-loader/pom.xml Outdated Show resolved Hide resolved
hugegraph-tools/pom.xml Outdated Show resolved Hide resolved
hugegraph-loader/pom.xml Show resolved Hide resolved
@imbajin imbajin changed the title fix shell execute permission fix(dist): bin package lack execute permission Dec 8, 2022
@coderzc coderzc requested a review from imbajin December 9, 2022 06:52
Comment on lines -190 to -194
<directory>output</directory>
<sourceDirectory>src/main/java</sourceDirectory>
<!--<testSourceDirectory>src/test/java</testSourceDirectory>-->
<outputDirectory>output/classes</outputDirectory>
<!--<testOutputDirectory>output/test-classes</testOutputDirectory>-->
Copy link
Member

Choose a reason for hiding this comment

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

@javeme ensure is it useless now?

@imbajin imbajin merged commit 70a2100 into apache:master Dec 9, 2022
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