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

Send nil to ActiveSupport#compile which now requires a parameter #807

Merged
merged 3 commits into from
Aug 11, 2023
Merged

Send nil to ActiveSupport#compile which now requires a parameter #807

merged 3 commits into from
Aug 11, 2023

Conversation

gucki
Copy link
Contributor

@gucki gucki commented Aug 11, 2023

Fixes #787

@jkowens jkowens merged commit e85a287 into zdennis:master Aug 11, 2023
14 checks passed
@jkowens
Copy link
Collaborator

jkowens commented Aug 11, 2023

Looks good, thank you!

@gucki
Copy link
Contributor Author

gucki commented Aug 16, 2023

@jkowens You are welcome! Would you mind releasing a new version with the fix? :)

@jkowens
Copy link
Collaborator

jkowens commented Aug 21, 2023

@gucki sorry for the delay, this has been released in v1.5.0 🎉

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.

Upcoming breakage with next Rails release
3 participants