Skip to content

Require logger for concurrent-ruby 1.3.5#2525

Merged
dblock merged 4 commits into
ruby-grape:masterfrom
ericproulx:fix_require_logger
Jan 20, 2025
Merged

Require logger for concurrent-ruby 1.3.5#2525
dblock merged 4 commits into
ruby-grape:masterfrom
ericproulx:fix_require_logger

Conversation

@ericproulx

@ericproulx ericproulx commented Jan 19, 2025

Copy link
Copy Markdown
Contributor

This breaks ActiveSupport 6.1 and 7.0. Requiring logger before active_support fixes the issue

@grape-bot

grape-bot commented Jan 19, 2025

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

Generated by 🚫 Danger

@ericproulx ericproulx marked this pull request as ready for review January 19, 2025 20:48
@ericproulx ericproulx requested a review from dblock January 19, 2025 20:58
@dblock dblock merged commit 926c28d into ruby-grape:master Jan 20, 2025
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