Skip to content

Conversation

@julien-f
Copy link
Contributor

BREAKING: as chartist is no longer a solid dependency, it's necessary for react-chartist users to depend on it directly.

react-chartist version should be bumped to 0.11.0.

Replace both #42 and #43.

BREAKING: as chartist is no longer a solid dependency, it's necessary for react-chartist users to depend on it directly.

react-chartist version should be bumped to 0.11.0.
@julien-f
Copy link
Contributor Author

@fraserxu It would be nice if it could land quickly :)

As soon as we have make sure it does not create issues of course ^^

package.json Outdated
"watchify": "^2.2.1"
},
"peerDependencies": {
"chartist": "^0.10.1"
Copy link
Owner

Choose a reason for hiding this comment

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

Can you move it to the existing peerDependencies above?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, my bad :)

@fraserxu
Copy link
Owner

Thanks @julien-f I'll merge it and do a new release.

@fraserxu fraserxu merged commit 6815527 into fraserxu:master Nov 15, 2016
@fraserxu
Copy link
Owner

@julien-f published to v0.11.0

This was referenced Nov 15, 2016
@julien-f
Copy link
Contributor Author

Thank you so much, I hoped it won't cause any issues! 😛

In any case, should not be a big deal as it's a new major version :)

@julien-f julien-f deleted the patch-1 branch November 15, 2016 11:56
@fraserxu
Copy link
Owner

0.11.0 is a minor version though

@julien-f
Copy link
Contributor Author

Yes, but I meant not compatible with ^0.10, it's kind of a major for the zero line ^^

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