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

Make it work with Ruby 3.1.4 #3

Closed
wants to merge 7 commits into from
Closed

Make it work with Ruby 3.1.4 #3

wants to merge 7 commits into from

Conversation

etlund
Copy link

@etlund etlund commented Nov 8, 2023

I'm not sure this covers everything, but I was able to get the demo app to work with ruby 3.1.4.

* commit '373802cdf6aed5d94d956f962a23507641a90191':
  Updated dependencies
  bump Ruby version
  Version bump
  Added ability to assign data attributes to modal content
  Support for Ruby 3.3

# Conflicts:
#	.tool-versions
@etlund etlund closed this Feb 2, 2024
@nzifnab
Copy link

nzifnab commented Apr 29, 2024

Was this meant to be closed without merging? I notice that this includes the same fix I made for the action controller loading in #10, which definitely wasn't in main.

@etlund
Copy link
Author

etlund commented May 1, 2024

I closed it because I wasn't sure this was all that was required to get it compatible with ruby 3.1.4, and I updated my app to ruby 3.2.2 before digging any deeper.

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.

2 participants