Skip to content
This repository has been archived by the owner on Jul 10, 2024. It is now read-only.

SUBMARINE-640. Remove workbench (vue) #416

Closed
wants to merge 1 commit into from

Conversation

kevin85421
Copy link
Member

@kevin85421 kevin85421 commented Sep 29, 2020

What is this PR for?

Workbench (vue) is outdated, and thus it should be removed from this repository.

What type of PR is it?

[Improvement]

Todos

workbench-web-ng --> workbench-web
Update some documents

What is the Jira issue?

https://issues.apache.org/jira/browse/SUBMARINE-640

How should this be tested?

https://travis-ci.org/github/kevin85421/hadoop-submarine/builds/731153069

Screenshots (if appropriate)

Questions:

  • Does the licenses files need update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@kevin85421
Copy link
Member Author

Please help me review the following files:
(1) .gitignore
(2) .travis.yml
(3) pom.xml
(4) submarine-dist/src/assembly/src-distribution.xml
(5) submarine-workbench/pom.xml

Thanks a lot!

Copy link
Member

@xunliu xunliu left a comment

Choose a reason for hiding this comment

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

LGTM
@kevin85421 Please create new ticket rename workbench-web-ng to workbench-web.

@asfgit asfgit closed this in 69977a4 Sep 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants