Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

fix: change ^ to ~ on 0.x.x deps #1345

Merged
merged 1 commit into from
May 9, 2018
Merged

fix: change ^ to ~ on 0.x.x deps #1345

merged 1 commit into from
May 9, 2018

Conversation

dryajov
Copy link
Member

@dryajov dryajov commented May 8, 2018

No description provided.

@ghost ghost assigned dryajov May 8, 2018
@ghost ghost added the status/in-progress In progress label May 8, 2018
Copy link
Contributor

@fsdiogo fsdiogo left a comment

Choose a reason for hiding this comment

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

LGTM

@alanshaw
Copy link
Member

alanshaw commented May 9, 2018

hey @dryajov what's the reason for this? I was under the impression that ^ has the same semantics for 0.x versions as ~. Am I missing something?

https://docs.npmjs.com/misc/semver#tilde-ranges-123-12-1

@fsdiogo
Copy link
Contributor

fsdiogo commented May 9, 2018

Hey @alanshaw, the js-ipfs guidelines states that:

Use ~ for everything below 1.0.0 and ^ for everything above 1.0.0

~ is for patches, ^ is for minors.

@alanshaw
Copy link
Member

alanshaw commented May 9, 2018

Ok, my bad! LGTM

@alanshaw
Copy link
Member

alanshaw commented May 9, 2018

For next time @dryajov please can we have sign off on commits? https://github.com/ipfs/community/blob/master/js-code-guidelines.md#sign-off-on-commits

@alanshaw alanshaw merged commit de95989 into master May 9, 2018
@ghost ghost removed the status/in-progress In progress label May 9, 2018
@alanshaw alanshaw deleted the fix/semantic-v branch May 9, 2018 11:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants