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

material-ui@0.17 #1136

Merged
merged 1 commit into from Feb 20, 2017
Merged

material-ui@0.17 #1136

merged 1 commit into from Feb 20, 2017

Conversation

feross
Copy link
Member

@feross feross commented Feb 15, 2017

Tested and everything seems to work alright with the new version.

Tested and everything seems to work alright with the new version.
@codealchemist
Copy link
Contributor

Win7 on VirtualBox:
image

image

image

macOS Sierra 10.12.2
image

image

@codealchemist
Copy link
Contributor

@feross Should we change the package version to 0.19.0?
The rest looks good :)
Did you test it on Linux?

Cheers!

@feross
Copy link
Member Author

feross commented Feb 17, 2017

Thanks for the testing :) I already tested Mac, but a second pair of eyes doesn't hurt. 👍

Should we change the package version to 0.19.0?

Isn't the latest 0.17.0? At least that's what I see! https://www.npmjs.com/package/material-ui

@codealchemist
Copy link
Contributor

I was referring to the version of WebTorrent Desktop.

It's currently 0.18.0 and I was wondering if after this changes it might get a bump on the minor.
But maybe it's ok not to update the version at all:

MAJOR version when you make incompatible API changes,
MINOR version when you add functionality in a backwards-compatible manner, and
PATCH version when you make backwards-compatible bug fixes.

What are your thoughts on it?
Thanks!

@codealchemist
Copy link
Contributor

I'm also thinking about the changelog.
The plan is to include other PRs and then release version 0.19.0 right?
Thx!

Copy link
Contributor

@codealchemist codealchemist left a comment

Choose a reason for hiding this comment

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

Looking good! Tested on OSX and Windows 7.

@feross feross merged commit 972203d into master Feb 20, 2017
@feross feross deleted the f/material-ui branch February 20, 2017 11:10
@feross
Copy link
Member Author

feross commented Feb 20, 2017

I was referring to the version of WebTorrent Desktop.

Ahh, yes. We usually bump the version in package.json right before doing the release. That's so that there's a single commit like 0ee7e80 where the version changes. Also, we add a git tag there too, so git checkout v0.18.0 checks you out to the specified version. 👍

The plan is to include other PRs and then release version 0.19.0 right?

Yes, I'd love to catch up on all the outstanding PRs, and do a release with all those changes!

@codealchemist
Copy link
Contributor

Cool! Thanks for the explanation dude!

@lock lock bot locked as resolved and limited conversation to collaborators May 10, 2018
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.

None yet

2 participants