-
Notifications
You must be signed in to change notification settings - Fork 463
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
Prepping next releases #745
Comments
Hi @mathieujobin , I just tested against master@85aa3b5 and it works for me in a rails 7 env. Thanks! Those Rails 4.2 problems are really strange... I won't be sad once those are dropped from the test matrix. Generally: LGTM! |
0.6.0 with Ruby 3 fixes only sound just fine, yeah. I'm okay with dropping Rails 4 support, it's really quite old and probably not supported anymore :/ BTW, @mathieujobin, do you want rubygems perms, so you're able to create gems or you want me to create them after you've pushed the tags here? I somehow missed that... |
Yes I can build and push directly if you give me access. https://rubygems.org/profiles/mathieu Is my profile. I think you can invite by username?
Otherwise I would like to give you my email privately... |
I've sent the invite :) |
@mathieujobin I won't have time for the mentioned rails 7 thingy as it didn't work as smooth as I expected yesterday and I don't understand the error yet. |
I will soon |
updated changelog and bump version for next release 5b925fc |
next inline for review @themilkman @ofedoren I can release tomorrow |
We're good for now |
Actually, this is what I am thinking to include next I could bump to 0.8.0 as it fix a security issue and potentially has visual changes. opinion? |
I'm okay with that, maybe we could even CP it to the stable and release 0.5.21 since it's a security fix. My only concern is if the license is the same (probably it is, just to be sure) |
Couldn't try the PR ATM (will try to locally build with the PR and do a quick optical compare later) but I checked the license headers in the diffs - they didn't change. |
In my docs I could observe minor visible differences using a rebased PR branch of #708 but generally it looked even better. Nothing seems to be broken so I think it is fine. 👍 |
Thanks for testing that out! I know it's a bit much to ask, but could you provide screenshot of what have changed, just so more people are aware? |
This is great help @themilkman |
opinions on #691 ? |
My bad, I can do it fine... |
... continuing conversations from #743
@ofedoren @themilkman I would release current master as 0.6.0 as it only constains ruby 3 fixes and the removal of unnecessary dependencies.
#737 could go along if all tests pass, or in 0.6.1
I am curious to understand the issue that #744 fixes
but if all tests pass, it seems harmless to include soon.
similar with #732
this bug fix rings a bell... #733
but 3 tests are failing on rails 4.2
so this would go into 1.0 where I would remove support for
< 5.2
unless the author figure out the tests
The text was updated successfully, but these errors were encountered: