Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Tailor "gulp missing" message when yarn.lock file exists #158

Merged
merged 1 commit into from May 20, 2020

Conversation

bdukes
Copy link
Contributor

@bdukes bdukes commented Apr 6, 2018

This was separated from #155, and builds on top of it

@phated
Copy link
Member

phated commented Apr 20, 2019

@bdukes can you rebase this?

@bdukes
Copy link
Contributor Author

bdukes commented Apr 22, 2019

@phated I've rebased, let me know if you need anything else!

@phated phated merged commit a41ed3f into gulpjs:master May 20, 2020
@phated
Copy link
Member

phated commented May 20, 2020

@bdukes I'm really sorry this has taken so long to get landed. I know it's hard to stay motivated when we take awhile to respond. I'm going to get this out tonight once CI finishes.

I'm also considering this a "fix" since we are making an incorrect recommendation in the yarn scenario.

@bdukes bdukes deleted the yarn-message branch May 20, 2020 13:23
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.

None yet

2 participants