Skip to content

Add distTag option#151

Merged
sindresorhus merged 4 commits intosindresorhus:masterfrom
LitoMore:add-dist-tag-option
Apr 7, 2019
Merged

Add distTag option#151
sindresorhus merged 4 commits intosindresorhus:masterfrom
LitoMore:add-dist-tag-option

Conversation

@LitoMore
Copy link
Copy Markdown
Contributor

Resolve #150

@LitoMore
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

LitoMore commented Mar 7, 2019

@SBoudrias ping

Comment thread .travis.yml Outdated
- '4'
- '8'
- '10'
- 'node'
Copy link
Copy Markdown
Owner

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.

Comment thread index.js Outdated
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
Copy Markdown
Owner

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
Copy Markdown
Owner

Can you fix the merge conflict?

@LitoMore
Copy link
Copy Markdown
Contributor Author

@sindresorhus Code updated.

@sindresorhus sindresorhus merged commit c8faa84 into sindresorhus:master Apr 7, 2019
@LitoMore LitoMore deleted the add-dist-tag-option branch April 8, 2019 02:41
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.

2 participants