Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Installing Intern using NPM #10

Closed
vladikoff opened this issue Apr 27, 2013 · 12 comments
Closed

Installing Intern using NPM #10

vladikoff opened this issue Apr 27, 2013 · 12 comments
Assignees
Labels
enhancement A new or improved feature
Milestone

Comments

@vladikoff
Copy link
Member

We've discussed this a bit before with @csnover earlier.
Would adding Intern to NPM make the framework easier and more accessible to the users?

@bitpshr
Copy link
Contributor

bitpshr commented Apr 27, 2013

I think this is a good idea.

I have an open pull request to make an updated Intern Grunt task part of this repository, so being able to distribute everything as a single package via NPM for world domination of JavaScript testing would be a good thing.

+1 for distributing Intern via NPM regardless of the route we take with Grunt.

@vladikoff
Copy link
Member Author

I saw the Grunt Task PR, really awesome! can't wait to use it :)

@csnover
Copy link
Member

csnover commented Apr 27, 2013

Yes, it is definitely planned to have this registered as an npm package. The install instructions will be updated when it is.

@csnover
Copy link
Member

csnover commented May 1, 2013

There is now a registered npm package, but the documentation is not yet updated.

@ghost ghost assigned csnover May 1, 2013
@phated
Copy link
Contributor

phated commented May 1, 2013

Hooray! Thanks Colin!

@milesj
Copy link
Contributor

milesj commented May 2, 2013

Awesome, was going to ask for this. However, when trying to install intern through NPM, I keep receiving this error... which is weird because the repo looks fine.

npm WARN `git config --get remote.origin.url` returned wrong result (git://githu
b.com/csnover/sauce-connect-launcher.git) undefined
npm WARN `git config --get remote.origin.url` returned wrong result (git://githu
b.com/csnover/sauce-connect-launcher.git) undefined
npm ERR! git clone git://github.com/csnover/sauce-connect-launcher.git undefined

npm ERR! git clone git://github.com/csnover/sauce-connect-launcher.git undefined

@csnover
Copy link
Member

csnover commented May 2, 2013

Do you have Git installed?

@milesj
Copy link
Contributor

milesj commented May 2, 2013

Yeah that was my mistake, I fixed it already. I have git installed but cygwin couldn't reference it.

@altano
Copy link

altano commented May 6, 2013

So I'm confused. This is an open issue for milestone 1.1 and is still open, but intern appears to be available and installable via npm as version 1.0. But the "Using Intern with Grunt" page says:

Install Intern and load the Grunt task into your Gruntfile using grunt.loadTasks('path/to/intern/grunt'); (Intern 1.0.0) or grunt.loadNpmTasks('intern') (Intern 1.1+).

... but loadNpmTasks() fails with Intern 1.0 currently on npm.

So, what does it mean to support loadNpmTasks() with Intern 1.1 if that's not on npm yet? Was that documentation just jumping the gun on the release?

I'm assuming that if I want to use Intern with Grunt right now, I can't use npm and I should just download via git and call loadTasks() pointing at the intern/grunt. Right?

This stuff all looks pretty cool by the way, so thanks to all of you for working on this.

@csnover
Copy link
Member

csnover commented May 6, 2013

Documentation is always updated to reflect the current state of development. Version numbers are put in the documentation so you know what’s supported or not. The copy of Intern currently available via npm exists to prevent package name squatting and to perform testing of installations via npm and is not yet an officially supported installation method. Official installation support via npm will first appear in 1.1. If you are using Grunt with Intern 1.0, you need to load the tasks as per the 1.0 instructions (loadTasks).

@altano
Copy link

altano commented May 7, 2013

That makes sense. I'm still having trouble but I'll start a new issue to discuss it.

@HelloGrayson
Copy link

Mega +1

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement A new or improved feature
Projects
None yet
Development

No branches or pull requests

7 participants