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

Log out yarn version #426

Merged
merged 2 commits into from Jun 13, 2017
Merged

Log out yarn version #426

merged 2 commits into from Jun 13, 2017

Conversation

jmorrell
Copy link
Contributor

@jmorrell jmorrell commented Jun 9, 2017

Fixes #423

Cases:

Not using Yarn:
hyper 2017-06-09 10-43-45

yarn.lock but no version:
hyper 2017-06-09 10-42-56

yarn.lock with version:
hyper 2017-06-09 10-43-26

Copy link
Contributor

@hunterloftis hunterloftis left a comment

Choose a reason for hiding this comment

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

Thanks for the bite-sized changeset and screenshots.

lgtm.

@jmorrell
Copy link
Contributor Author

jmorrell commented Jun 9, 2017

Tiny diffs FTW

@kennethreitz
Copy link

with version is unaligned in the screenshot

@kennethreitz
Copy link

an extra space

bin/compile Outdated
@@ -93,6 +93,9 @@ install_bins() {
echo "engines.node (package.json): ${node_engine:-unspecified}"
fi
echo "engines.npm (package.json): ${npm_engine:-unspecified (use default)}"
if $YARN; then
echo "engines.yarn (package.json): ${yarn_engine:-unspecified (use default)}"

Choose a reason for hiding this comment

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

i think we want one fewer spaces here

@hunterloftis
Copy link
Contributor

Good eye, didn't even notice that.

@jmorrell
Copy link
Contributor Author

jmorrell commented Jun 9, 2017

hyper 2017-06-09 11-48-57

Fixed!

@jmorrell jmorrell merged commit e00996d into master Jun 13, 2017
@jmorrell jmorrell deleted the log-yarn branch June 13, 2017 05:36
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

3 participants