Skip to content

Conversation

@spadgett
Copy link
Contributor

@petervo
Copy link

petervo commented Jul 31, 2017

Can we bump the package.json version as well? Maybe the changes from #23 would be good to include as well.

@spadgett
Copy link
Contributor Author

I was going to, but didn't see a version in package.json

@petervo
Copy link

petervo commented Jul 31, 2017

I think that's what #23 was adding

@spadgett
Copy link
Contributor Author

I don't mind adding it. @petervo were you saying to add those changes to this PR or wait until #23 merges? Would be good to cut a release so that we can pick up the fix in #28.

@stefwalter Any preference?

@petervo
Copy link

petervo commented Jul 31, 2017

Either way works for me. I just don't have perms to merge anything, that's why I'm bringing it up

@spadgett
Copy link
Contributor Author

@petervo OK pulled in the package.json changes from #23

@petervo
Copy link

petervo commented Aug 1, 2017

LGTM

@spadgett spadgett merged commit 8470a02 into kubernetes-ui:master Aug 1, 2017
@spadgett spadgett deleted the bump-to-v2.0.2 branch August 1, 2017 16:25
@spadgett spadgett mentioned this pull request Aug 1, 2017
@stefwalter
Copy link
Contributor

I gave both @petervo and @spadgett access to the NPM package

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.

3 participants