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

Rails 7 - uninitialized constant ActiveSupport::XmlMini::IsolatedExecutionState #2208

Merged
merged 3 commits into from
Dec 27, 2021

Conversation

ericproulx
Copy link
Contributor

Hello,

The PR will fixed the uninitialized constant ActiveSupport::XmlMini::IsolatedExecutionState for ActiveSupport 7. It also fixes the rubocop.

Merry Xmas! 🎄

…ort::VERSION::MAJOR > 6

Add rails-7 appraisals
Fix rubocop Performance/StringIdentifierArgument
@grape-bot
Copy link

grape-bot commented Dec 25, 2021

1 Warning
⚠️ There're library changes, but not tests. That's OK as long as you're refactoring existing code.

Generated by 🚫 Danger

Copy link
Member

@dblock dblock left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

Add Rails 7 to CI, in https://github.com/ruby-grape/grape/blob/master/.github/workflows/test.yml#L40

Add CHANGELOG saying "Added Rails 7 support"

Add CHANGELOG
@dblock dblock merged commit 8c809fa into ruby-grape:master Dec 27, 2021
@dm1try
Copy link
Member

dm1try commented Dec 27, 2021

The PR will fixed the uninitialized constant ActiveSupport::XmlMini::IsolatedExecutionState for ActiveSupport 7.

it has already been fixed in master by #2206
the important context: rails/rails#43852 (comment)

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.

4 participants