Skip to content

Upgrade for Rails 7.1 - #203

Open
simpl1g wants to merge 4 commits into
jmazzi:masterfrom
simpl1g:upgrade-for-rails7-1
Open

Upgrade for Rails 7.1#203
simpl1g wants to merge 4 commits into
jmazzi:masterfrom
simpl1g:upgrade-for-rails7-1

Conversation

@simpl1g

@simpl1g simpl1g commented Apr 18, 2024

Copy link
Copy Markdown

I updated public/private keys, I'm not sure why they stopped working, maybe they expired. So I generated new with 50 years expiration.

@simpl1g

simpl1g commented May 8, 2024

Copy link
Copy Markdown
Author

@jmazzi Hi, sorry to bother you, any chances to review and release new version of gem? Tests are passing locally

@khrvi

khrvi commented Jun 4, 2024

Copy link
Copy Markdown

@jmazzi could you merge it if it looks good for you

@ByteDecoder

Copy link
Copy Markdown

@jmazzi please release the Rails 7.0.x gem version (right now only works with Rails 7 but not for 7.0.x) before this Rails 7.1 gem version

@johan-smits

Copy link
Copy Markdown
Contributor

This MR will support all 7.x versions. Is this desired that it will already support the upcoming Rails 7.2 release?

@simpl1g

simpl1g commented Jun 8, 2024

Copy link
Copy Markdown
Author

@johan-smits thanks, makes sense, downgraded dependency to exclude 7.2

@tkling

tkling commented Sep 12, 2024

Copy link
Copy Markdown

Bernie Sanders sitting in a foldable chair

@jmazzi bumping this now that it's idled for a few months 🙏 any chance you could review please?

@simpl1g

simpl1g commented Sep 16, 2024

Copy link
Copy Markdown
Author

Also prepared PR for Rails 7.2
simpl1g#2

@ByteDecoder

Copy link
Copy Markdown

@simpl1g do you have permission to merge this branch?

@simpl1g

simpl1g commented Jul 30, 2025

Copy link
Copy Markdown
Author

@simpl1g do you have permission to merge this branch?

Sorry, I don't have such permission, I would love to merge this :)

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.

5 participants