Skip to content

Commit

Permalink
added note to README on browser build target
Browse files Browse the repository at this point in the history
  • Loading branch information
Caolan McMahon committed Nov 21, 2010
1 parent cd9bd21 commit e6483f4
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions README.md
Expand Up @@ -279,6 +279,10 @@ why nodeunit does not currently provide this out of the box.

In the example above, the tests will run when the page is loaded.

The browser-version of nodeunit.js is created in dist/browser when you do, 'make
browser'. You'll need [UglifyJS](https://github.com/mishoo/UglifyJS) installed in
order for it to automatically create nodeunit.min.js.


Adding nodeunit to Your Projects
--------------------------------
Expand Down

3 comments on commit e6483f4

@Sannis
Copy link
Contributor

@Sannis Sannis commented on e6483f4 Nov 21, 2010

Choose a reason for hiding this comment

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

You can use 'dependencies' property in package.json, if you want :-)

@caolan
Copy link
Owner

@caolan caolan commented on e6483f4 Nov 22, 2010

Choose a reason for hiding this comment

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

That might be a good idea, but I doubt people installing via npm will want to do a browser-build anyway. I think they would be more likely to check it out and do 'make browser'. I'm not sure of the best way to distribute the browser build atm, I might just check it into the repository...

@Sannis
Copy link
Contributor

@Sannis Sannis commented on e6483f4 Nov 22, 2010

Choose a reason for hiding this comment

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

Oh, in this case i agree with you.

Please sign in to comment.