Skip to content
This repository has been archived by the owner on Aug 11, 2022. It is now read-only.

add 'npm it' command #9443

Closed
wants to merge 3 commits into from
Closed

add 'npm it' command #9443

wants to merge 3 commits into from

Conversation

isaacs
Copy link
Contributor

@isaacs isaacs commented Aug 30, 2015

I keep wanting a command for npm i && npm t so I could do npm it.

Needs tests and docs, but leaving this here for my fellow CLI peops to weigh in on whether this is a good thing or an excessive thing.

@othiym23
Copy link
Contributor

Aaglaglgalgaglaglahaghaga I want this ALL THE TIME. It might confuse some people's muscle memory, but YES THIS IS A GOOD IDEA:+1::japanese_goblin::ski::sunflower:

@othiym23 othiym23 added this to the 2.x-next-next milestone Aug 31, 2015
@othiym23
Copy link
Contributor

Sounds good, we'll try to get this out next week if you want to put together some tests and docs!

@othiym23
Copy link
Contributor

(and let's go ahead and do this against both 2 and 3)

@zkat zkat modified the milestones: 2.x-next, 2.14.4 Sep 11, 2015
@zkat zkat self-assigned this Sep 11, 2015
@zkat
Copy link
Contributor

zkat commented Sep 11, 2015

Missing tests so I'm bumping until next week. Should be quick for me to write some basic ones for this.

@othiym23 othiym23 modified the milestones: 2.x-next-next, 2.x-next Sep 14, 2015
@zkat
Copy link
Contributor

zkat commented Sep 14, 2015

Woe is me, if only SOMEONE would be willing to write tests + docs for this ;)

(I can take care of it, but it'll be a bit before I have time)

@zkat zkat modified the milestones: 3.x-next-next, next-2 Sep 24, 2015
@iarna iarna modified the milestones: 3.x-next-next, 3.x-next Oct 7, 2015
@iarna
Copy link
Contributor

iarna commented Oct 19, 2015

I'm removing the milestone from this because, at this moment, writing tests for this (which is what's needed to land it) is not scheduled for this week or next.

@iarna iarna removed this from the 3.3.9 milestone Oct 19, 2015
@mgol
Copy link

mgol commented Oct 20, 2015

Sometimes pruning is also needed, would it be worthwhile to have a shortcut for:

npm prune && npm install && npm test

?

@othiym23
Copy link
Contributor

I don't think that's the dominant use case, @mzgol, and I'd like to keep this a simple, focused addition if we can.

@mgol
Copy link

mgol commented Oct 20, 2015

OK, fair enough.

@ashleygwilliams
Copy link
Contributor

documentation added.

@iarna
Copy link
Contributor

iarna commented Nov 5, 2015

Tests look good: 🐑

isaacs added a commit that referenced this pull request Nov 5, 2015
ashleygwilliams pushed a commit that referenced this pull request Nov 5, 2015
othiym23 added a commit that referenced this pull request Nov 5, 2015
@othiym23
Copy link
Contributor

othiym23 commented Nov 5, 2015

Rebased against release-3.4.0 and landed as 1ac3e08, bcb04f6, and b6c17dd. This was a real team effort. Thanks, everybody!

@othiym23 othiym23 closed this Nov 5, 2015
@othiym23 othiym23 deleted the npmit branch November 5, 2015 23:29
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants