Skip to content

Conversation

@JoshuaKGoldberg
Copy link
Contributor

This was originally discussed here: npm/npm#20741

Equivalent Yarn issue: yarnpkg/yarn#5867

This was originally discussed here: npm/npm#20741

Equivalent Yarn issue: yarnpkg/yarn#5867

> This is required, even if it seems like a stretch

I'm sorry, but I can't come up with alternatives to help with this problem.

Choose a reason for hiding this comment

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

Alternatives: List all scripts, keep current behavior.

@zkat
Copy link
Contributor

zkat commented May 31, 2018

I like this, tbh. We also already have a package we use for this sort of thing called meant that you can probably add to the implementation section. I believe it should work as you expect. Kudos to @watilde for it :)

@zkat
Copy link
Contributor

zkat commented May 31, 2018

Also, could you please create a corresponding thread for this in https://npm.community/c/ideas/rfcs?

watilde added a commit to watilde/npm that referenced this pull request Jun 3, 2018
Whenever npm run 'script' is unable to find 'script',
display a list with all possible available scripts.

Refs: npm/rfcs#9
Fixes: npm#20858
@JoshuaKGoldberg
Copy link
Contributor Author

@watilde
Copy link

watilde commented Jul 3, 2018

I also support this idea to save more time of debugging for developers. Then I made a patch at npm/npm#20879 and it's shipped already :)

We might want to close this issue as resolved.

Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

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

I mean we merged this, so ;p

I'll take care of wrapping up the RFC tomorrow. Thanks for this! Excited to have it, and thanks to @watilde for taking the initiative with implementing it!

@zkat zkat merged commit 9c84656 into npm:latest Jul 18, 2018
@JoshuaKGoldberg JoshuaKGoldberg deleted the run-suggestions branch July 18, 2018 20:47
@zkat zkat added the :ratified label Jan 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants