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

Constrain Zeitwerk to 2.6.5 #1059

Merged
merged 1 commit into from
Nov 15, 2022
Merged

Constrain Zeitwerk to 2.6.5 #1059

merged 1 commit into from
Nov 15, 2022

Conversation

andyw8
Copy link
Contributor

@andyw8 andyw8 commented Nov 11, 2022

I tried to resolve #1058 but couldn't figure out what's causing it, so for now I think it's better to set this constraint.

@andyw8 andyw8 marked this pull request as ready for review November 11, 2022 20:31
@andyw8 andyw8 requested a review from a team as a code owner November 11, 2022 20:31
@andyw8 andyw8 merged commit 9ef0f17 into main Nov 15, 2022
@andyw8 andyw8 deleted the andyw8/limit-to-zeitwerk-2.6.5 branch November 15, 2022 14:48
nelsonwittwer pushed a commit that referenced this pull request Nov 15, 2022
Co-authored-by: Andy Waite <andyw8@users.noreply.github.com>
nelsonwittwer added a commit that referenced this pull request Nov 18, 2022
* make session storage optional

* deprecate session storage

* logging

* cleaner deprecation strategy

* Clarify statement about packages (#1057)

Co-authored-by: Andy Waite <andyw8@users.noreply.github.com>

* Constrain Zeitwerk to 2.6.5 (#1059)

Co-authored-by: Andy Waite <andyw8@users.noreply.github.com>

* Update lib/shopify_api/context.rb

Co-authored-by: Andy Waite <13400+andyw8@users.noreply.github.com>

* Update lib/shopify_api/utils/session_utils.rb

Co-authored-by: Kevin O'Sullivan <56687600+mkevinosullivan@users.noreply.github.com>

* clean up deprecation logging

* raise error if trying to use deprecrated utils without session_storage

* throw error if misconfigured session_storage

* Update lib/shopify_api/context.rb

Co-authored-by: Kevin O'Sullivan <56687600+mkevinosullivan@users.noreply.github.com>

* change to more helpful upgrade instructions

* rubocop line length

* Update lib/shopify_api/context.rb

Co-authored-by: Kevin O'Sullivan <56687600+mkevinosullivan@users.noreply.github.com>

* Update lib/shopify_api/context.rb

Co-authored-by: Kevin O'Sullivan <56687600+mkevinosullivan@users.noreply.github.com>

* rubocop

Co-authored-by: Andy Waite <13400+andyw8@users.noreply.github.com>
Co-authored-by: Andy Waite <andyw8@users.noreply.github.com>
Co-authored-by: Kevin O'Sullivan <56687600+mkevinosullivan@users.noreply.github.com>
kaarelss added a commit to kaarelss/shopify-api-ruby that referenced this pull request Nov 29, 2022
…y-api-ruby into response_in_error_object

* 'response_in_error_object' of github.com:kaarelss/shopify-api-ruby:
  Added assertion that response is present in MaxHttpRetriesExceededError
  Added sorbet assertion
  Removed extra empty line
  ShopifyAPI::Clients::HttpResponse as argument for ShopifyAPI::Errors::HttpResponseError because code can be obtained from response as well as headers for custom retry logic
  Remove mentions of private apps (Shopify#1062)
  Fix ActiveSupport inflector dependency (Shopify#1063)
  Move Session Storage to `shopify_app` (Shopify#1055)
  Remove session argument from REST examples (Shopify#1064)
  Constrain Zeitwerk to 2.6.5 (Shopify#1059)
  Clarify statement about packages (Shopify#1057)
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.

Failure on Zeitwerk 2.6.5
2 participants