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

Ruby3 #104

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Ruby3 #104

wants to merge 11 commits into from

Conversation

anitagraham
Copy link
Contributor

Updates refinerycms-i18n to Ruby 3. There is a matching PR for refinerycms (refinery/refinerycms#3518)

Using Mobility 1.2.9.

Apart from that most changes are extremely minor.

refinerycms-i18n.gemspec Outdated Show resolved Hide resolved
@parndt
Copy link
Member

parndt commented May 12, 2023

Got CI running. Strangely, only Ruby 3.1 works and not 3.0 or 3.2 🤔

@halteren
Copy link

halteren commented Jun 29, 2023

Shouldn't line 29 of /lib/refinery/i18n-filter.rb be changed to

result.url !~ %r{^/(#{Refinery::Core.backend_route}|wymiframe)}

@anitagraham anitagraham reopened this Feb 14, 2024
@anitagraham
Copy link
Contributor Author

Closed and reopened to trigger new set of checks

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