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

SUBMARINE-402. Add tutorial for adding test cases and browsers to e2e with Angular#201

Closed
cchung100m wants to merge 1 commit intoapache:masterfrom
cchung100m:SUBMARINE-402
Closed

SUBMARINE-402. Add tutorial for adding test cases and browsers to e2e with Angular#201
cchung100m wants to merge 1 commit intoapache:masterfrom
cchung100m:SUBMARINE-402

Conversation

@cchung100m
Copy link
Copy Markdown

Hi @kevin85421

Following our discussion in #197, This PR is going to add the tutorial for the test cases adding and browsers configuration. I would appreciate if you can help to review it, thanks.

What is this PR for?

Add tutorial for adding test cases and browsers to e2e with Angular

What type of PR is it?

Documentation

What is the Jira issue?

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

How should this be tested?

https://travis-ci.com/cchung100m/submarine/builds/151553436

Questions:

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

@cchung100m cchung100m changed the title Add tutorial for adding test cases and browsers to e2e with Angular SUBMARINE-402. Add tutorial for adding test cases and browsers to e2e with Angular Mar 3, 2020
Copy link
Copy Markdown
Member

@kevin85421 kevin85421 left a comment

Choose a reason for hiding this comment

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

@cchung100m Thank you for your contribution! The pull request looks good to me, and I believe that it is ready to be merged.

Copy link
Copy Markdown
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.

Will merge if no more comments.

@asfgit asfgit closed this in 8f8740f Mar 5, 2020
@cchung100m cchung100m deleted the SUBMARINE-402 branch March 5, 2020 01:21
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.

3 participants