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

Version 1.0.0.pre.2 #731

Merged
merged 1 commit into from
Jul 28, 2020
Merged

Version 1.0.0.pre.2 #731

merged 1 commit into from
Jul 28, 2020

Conversation

RyanBrushett
Copy link
Contributor

What are you trying to accomplish with this PR?

  • Relax thor version constraint in k8s-deploy gemspec

How is this accomplished?

  • I picked >= 0.20 and < 2.0, seemed like a sensible enough range. Make sense?

@RyanBrushett RyanBrushett requested a review from dturn July 21, 2020 15:24
@RyanBrushett RyanBrushett force-pushed the relax-thor-version-requirement branch from 8fe356f to 8e14080 Compare July 21, 2020 15:25
CHANGELOG.md Show resolved Hide resolved
@timothysmith0609
Copy link
Contributor

I think you need to rename .shopify-build/kubernetes-deploy to krane for the pipeline

@RyanBrushett RyanBrushett force-pushed the relax-thor-version-requirement branch 5 times, most recently from 0ac1f4d to ea8f5f2 Compare July 27, 2020 14:39
Copy link
Contributor

@dirceu dirceu left a comment

Choose a reason for hiding this comment

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

LGTM

- Release Version 1.0.0.pre.2
- Relax thor version in k8s-deploy gemspec
- Relax googleauth restriction to ~> 0.9
- Pin K8s 1.16 CI to 1.16.12
@RyanBrushett RyanBrushett force-pushed the relax-thor-version-requirement branch from ea8f5f2 to e2704a6 Compare July 27, 2020 20:07
@@ -25,14 +25,14 @@ Gem::Specification.new do |spec|
spec.required_ruby_version = '>= 2.4.0'
spec.add_dependency("activesupport", ">= 5.0")
spec.add_dependency("kubeclient", "~> 4.3")
spec.add_dependency("googleauth", "~> 0.8.0")
spec.add_dependency("googleauth", "~> 0.8")
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dirceu I made a slight adjustment to match the googleauth dependency restriction with Krane's latest. You still 👍 ?

@RyanBrushett
Copy link
Contributor Author

The only failure is on the linter and it's angry about the name kubernetes-deploy which isn't going anywhere so merging 👍

@RyanBrushett RyanBrushett merged commit 646838b into 0.x Jul 28, 2020
@RyanBrushett RyanBrushett deleted the relax-thor-version-requirement branch July 28, 2020 14:01
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.

None yet

4 participants