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

Add distTag option #151

Merged
merged 4 commits into from
Apr 7, 2019
Merged

Conversation

LitoMore
Copy link
Contributor

Resolve #150

@LitoMore
Copy link
Contributor Author

LitoMore commented Feb 22, 2019

@sindresorhus Could you help me review why this CI failed (for the Node.js 6 job)?

Should I update the .travis.yml for higher Node.js version in this pull request?

@LitoMore
Copy link
Contributor Author

LitoMore commented Mar 7, 2019

@SBoudrias ping

.travis.yml Outdated
- '4'
- '8'
- '10'
- 'node'
Copy link
Member

Choose a reason for hiding this comment

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

Don't do unrelated changes.

index.js Outdated
@@ -36,7 +37,7 @@ class UpdateNotifier {
this.packageVersion = options.pkg.version;
this.updateCheckInterval = typeof options.updateCheckInterval === 'number' ? options.updateCheckInterval : ONE_DAY;
this.hasCallback = typeof options.callback === 'function';
this.callback = options.callback || (() => {});
this.callback = options.callback || (() => { });
Copy link
Member

Choose a reason for hiding this comment

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

Don't do unrelated changes.

@sindresorhus
Copy link
Member

Can you fix the merge conflict?

@LitoMore
Copy link
Contributor Author

@sindresorhus Code updated.

@sindresorhus sindresorhus merged commit c8faa84 into yeoman:master Apr 7, 2019
@LitoMore LitoMore deleted the add-dist-tag-option branch April 8, 2019 02:41
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants