Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

use ensure_loaded! with mixin objects #1468

Merged
merged 1 commit into from
Apr 8, 2023

Conversation

ccutrer
Copy link
Contributor

@ccutrer ccutrer commented Nov 14, 2022

This handles the case of a module including itself into the recipient, defined a different file

Completed Tasks

  • I have read the [Contributing Guide][contrib].
  • The pull request is complete (implemented / written).
  • Git commits have been cleaned up (squash WIP / revert commits).
  • I wrote tests and ran bundle exec rake locally (if code is attached to PR).

this handles the case of a module including itself into the recipient, defined
in a different file
@lsegal lsegal merged commit 2d6d89f into lsegal:main Apr 8, 2023
@lsegal
Copy link
Owner

lsegal commented Apr 8, 2023

Sorry about the delay. Thanks for the PR!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants