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

Remove top header, add test commands #1029

Merged
merged 4 commits into from Jan 10, 2019

Conversation

Projects
None yet
4 participants
@RVMendoza
Copy link
Collaborator

RVMendoza commented Jan 10, 2019

What does this PR do?

Remove unneeded top header, add test commands so user knows what commands to run to check for coverage

@RVMendoza RVMendoza self-assigned this Jan 10, 2019

@yodasw16
Copy link
Member

yodasw16 left a comment

Adding the test commands here is a great addition. I feel like at one point you had to be in a specific directory to run tests. Can these commands be run from the project root?

@yodasw16 yodasw16 merged commit 8a02d70 into staging Jan 10, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
deploy/netlify Deploy preview ready!
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment