Actually remove ospec from the main code base (#2786)
* Actually remove ospec from the main code base (it is still used as a dev-dependency) * fix a copy error Co-authored-by: Barney Carroll <barney.carroll@gmail.com> Co-authored-by: Barney Carroll <barney.carroll@gmail.com>
This commit is contained in:
parent
ea680dc917
commit
c0a9c21c09
12 changed files with 4 additions and 2051 deletions
|
|
@ -49,19 +49,17 @@ To send a pull request:
|
|||
|
||||
## I'm submitting a PR. How do I run tests?
|
||||
|
||||
Assuming you have forked this repo, you can open the `index.html` file in a module's `tests` folder and look at console output to see only tests for that module, or you can run `ospec/bin/ospec` from the command line to run all tests.
|
||||
After having run `npm install` (a one-time operation), run `npm run test` from the command line to run all tests.
|
||||
|
||||
While testing, you can modify a test to use `o.only(description, test)` instead of `o(description, test)` if you wish to run only a specific test to speed up your debugging experience. Don't forget to remove the `.only` after you're done!
|
||||
|
||||
There is no need to `npm install` anything in order to run the test suite, however NodeJS is required to run the test suite from the command line. You do need to `npm install` if you want to lint or get a code coverage report though.
|
||||
|
||||
|
||||
|
||||
## How do I build Mithril.js?
|
||||
|
||||
If all you're trying to do is run examples in the codebase, you don't need to build Mithril.js, you can just open the various html files and things should just work.
|
||||
|
||||
To generate the bundled file for testing, run `npm run dev` from the command line. To generate the minified file, run `npm run build`. There is no need to `npm install` anything, but NodeJS is required to run the build scripts.
|
||||
To generate the bundled file for testing, run `npm run dev` from the command line. To generate the minified file, run `npm run build`.
|
||||
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -14,4 +14,4 @@
|
|||
- Optional
|
||||
- [Stream](stream.md)
|
||||
- Tooling
|
||||
- [Ospec](https://github.com/MithrilJS/mithril.js/blob/master/ospec)
|
||||
- [Ospec](https://github.com/MithrilJS/ospec)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue