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

Update the gemspec for Krane 1.0 #631

Merged
merged 1 commit into from
Nov 18, 2019
Merged

Update the gemspec for Krane 1.0 #631

merged 1 commit into from
Nov 18, 2019

Conversation

dturn
Copy link
Contributor

@dturn dturn commented Nov 14, 2019

What are you trying to accomplish with this PR?
One of the last steps for releasing Krane 1.0.0 is updating the gemspec.

How is this accomplished?
Update the gemspec and use the existing description from https://rubygems.org/gems/krane

There is no rubygem rename ability, its just create a new one: https://help.rubygems.org/discussions/questions/104-how-should-i-handle-the-rename-of-a-gem

Shipit will automatically figure out the new gemspec.

What could go wrong?
Not much, worst case we quickly yank 1.0.0 while we fix the issue and create it again.

@dturn dturn requested a review from a team November 14, 2019 23:04
@dturn dturn mentioned this pull request Nov 14, 2019
8 tasks
spec.description = spec.summary
spec.homepage = "https://github.com/Shopify/kubernetes-deploy"
spec.homepage = "https://github.com/Shopify/krane"
Copy link
Contributor

Choose a reason for hiding this comment

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

Just to confirm, after the release are we going to rename the kubernetes-deploy to krane but are going to keep kubernetes-deploy repo around just with a README pointing to krane repo?

Copy link
Contributor Author

@dturn dturn Nov 15, 2019

Choose a reason for hiding this comment

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

We're going to rename the repo which should cause github to automatically redirect people. We'll also make sure the top of the readme mentions the rename and points to the migration guide

Copy link
Contributor

@douglas douglas left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@douglas douglas requested a review from a team November 15, 2019 16:53
Copy link
Contributor

@RyanBrushett RyanBrushett left a comment

Choose a reason for hiding this comment

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

This is exciting

@tsontario
Copy link

Can/do we document the yanking process anywhere? We've had a few releases go out that should be yanked (0.31.0 most recently) but I have no clue how to do this

@dturn
Copy link
Contributor Author

dturn commented Nov 16, 2019

Can/do we document the yanking process anywhere?

We don't. And as far as I know we don't have any automation around it either.

@dturn dturn changed the title [Do Not Merge] Update the gemspec for Krane 1.0 Update the gemspec for Krane 1.0 Nov 18, 2019
@dturn dturn merged commit 74b13d3 into master Nov 18, 2019
@dturn dturn deleted the gemspec-rename branch November 18, 2019 03:48
@ghost ghost added the krane [ProdX-GSD] label Nov 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants