-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
unit testing plugin info updated #2459
Conversation
src/app/tabs/test-tab.js
Outdated
<li> To write tests, visit our <a href="https://remix-ide.readthedocs.io/en/latest/unittesting.html#write-tests" target="_blank"> documentation </a></li> | ||
<li> To run tests, select file(s) and click on "Run Tests" button</li> | ||
</ol> | ||
To run unit tests in your Continuous Integration & as CLI, use the stand alone NPM module <a href="https://www.npmjs.com/package/remix-tests" target="_blank">remix-tests</a>.<br/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd like that the next line "To get support ..." is in a p tag - so the spacing is bigger - so remove the br and wrap it in a p tag
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or to save space - put the 2 sentences that start with To... in the same paragraph.
src/app/tabs/test-tab.js
Outdated
<li> To write tests, visit our <a href="https://remix-ide.readthedocs.io/en/latest/unittesting.html#write-tests" target="_blank"> documentation </a></li> | ||
<li> To run tests, select file(s) and click on "Run Tests" button</li> | ||
</ol> | ||
To run unit tests in your Continuous Integration & as CLI, use the stand alone NPM module <a href="https://www.npmjs.com/package/remix-tests" target="_blank">remix-tests</a>.<br/> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also here's a very small edit on this line:
To run unit tests in your Continuous Integration and as a CLI, use the stand alone NPM module
No description provided.