Skip to content

change npm run build:combined to npm run build:release#1504

Merged
lenboo merged 1 commit intoapache:devfrom
lgcareer:dev-ui-rel
Dec 18, 2019
Merged

change npm run build:combined to npm run build:release#1504
lenboo merged 1 commit intoapache:devfrom
lgcareer:dev-ui-rel

Conversation

@lgcareer
Copy link
Copy Markdown
Contributor

Tips

What is the purpose of the pull request

change npm run build:combined to npm run build:release

Brief change log

change npm run build:combined to npm run build:release

  • Add maven-checkstyle-plugin to root pom.xml

Verify this pull request

cd dolphinscheduler-ui execute npm run build:release

@codecov-io
Copy link
Copy Markdown

Codecov Report

Merging #1504 into dev will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff          @@
##             dev   #1504   +/-   ##
=====================================
  Coverage   2.77%   2.77%           
=====================================
  Files        272     272           
  Lines      13598   13598           
  Branches    2158    2158           
=====================================
  Hits         378     378           
  Misses     13201   13201           
  Partials      19      19

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 da1afb7...408347a. Read the comment docs.

Copy link
Copy Markdown
Contributor

@qiaozhanwei qiaozhanwei left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Copy Markdown
Contributor

@lenboo lenboo left a comment

Choose a reason for hiding this comment

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

+1

@lenboo lenboo merged commit 5404ff2 into apache:dev Dec 18, 2019
@lgcareer lgcareer deleted the dev-ui-rel branch December 3, 2020 07:18
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.

4 participants