Skip to content
This repository was archived by the owner on Feb 25, 2025. It is now read-only.

updating the documentation after integration_test migration #21077

Merged
merged 4 commits into from
Sep 11, 2020

Conversation

nturgut
Copy link
Contributor

@nturgut nturgut commented Sep 10, 2020

updating the documentation for running integration tests

  • update after migration to integration_test
  • adding more details on running with different browsers

@nturgut nturgut requested a review from ditman September 10, 2020 20:26
@nturgut nturgut requested a review from mdebbar September 10, 2020 20:26
@nturgut
Copy link
Contributor Author

nturgut commented Sep 10, 2020

@derek-j-pitman I updated the README, please refer to this one for running the tests.

Copy link
Member

@ditman ditman left a comment

Choose a reason for hiding this comment

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

Big improvement! I just left a few suggestions (simplify the commands as much as possible, until you explain new parameters and make the command more complex, that way people may learn how to build the full command they need, instead of relying on copy-paste)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants