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

Prepping next releases #745

Closed
mathieujobin opened this issue Mar 27, 2022 · 19 comments
Closed

Prepping next releases #745

mathieujobin opened this issue Mar 27, 2022 · 19 comments

Comments

@mathieujobin
Copy link
Collaborator

... 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

@themilkman
Copy link
Contributor

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.
Talking about the test matrix - I'll try to make a PR for rails 7 in order to get this more complete.

Generally: LGTM!

@ofedoren
Copy link
Contributor

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...

@mathieujobin
Copy link
Collaborator Author

mathieujobin commented Mar 28, 2022

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?

gem owner apipie-rails --add mathieu <-- 🙏

Otherwise I would like to give you my email privately...

@ofedoren
Copy link
Contributor

Is my profile. I think you can invite by username?

I've sent the invite :)

@themilkman
Copy link
Contributor

@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.
From my personal side side, I'd say: Release the kraken :)

@mathieujobin
Copy link
Collaborator Author

I will soon

@mathieujobin
Copy link
Collaborator Author

updated changelog and bump version for next release 5b925fc

@mathieujobin
Copy link
Collaborator Author

next inline for review @themilkman @ofedoren

#747

I can release tomorrow

@mathieujobin
Copy link
Collaborator Author

We're good for now

@mathieujobin
Copy link
Collaborator Author

Actually, this is what I am thinking to include next
#708

I could bump to 0.8.0 as it fix a security issue and potentially has visual changes.

opinion?

@mathieujobin mathieujobin reopened this Mar 29, 2022
@ofedoren
Copy link
Contributor

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)

@themilkman
Copy link
Contributor

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.

@themilkman
Copy link
Contributor

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. 👍

@ofedoren
Copy link
Contributor

ofedoren commented Apr 1, 2022

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. +1

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?

@mathieujobin
Copy link
Collaborator Author

This is great help @themilkman
This would go into 0.8.0, I could cherry-pick into stable for 0.5.21 but at the same time, I'd rather wait until there is demands for it...
there is one open bug against 0.7.0 right now, but otherwise, it could be all considered stable soon.
Giving it a few more months, though.

@themilkman
Copy link
Contributor

Hi, sorry, a bit busy but just found a few minutes for a quick optical impression what has changed:
Old:
pie_old
New:
pie_new
So stuff got bigger, gradient is gone and the "current" breadcrumb has a different color (as well as being blue when on the root). That's at least here on Firefox with a pull --rebase against master.

@mathieujobin
Copy link
Collaborator Author

opinions on #691 ?

@mathieujobin
Copy link
Collaborator Author

I'm looking at #689
Many positive votes towards it...

@ofedoren I think you have the ability to resolve the conflict on open PR.
Can I get that ability?

@mathieujobin
Copy link
Collaborator Author

My bad, I can do it fine...

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

No branches or pull requests

3 participants