Closed kreynen closed 1 year ago
I talked to @joelsteidl about this yesterday. While I'd like to see https://github.com/backdrop-contrib/simplesamlphp_auth/issues/4 and https://github.com/backdrop-contrib/simplesamlphp_auth/issues/5, resolving https://github.com/backdrop-contrib/simplesamlphp_auth/issues/2 was really the only blocker.
If this is running at multiple organizations with different SAML configurations (Stanford and the University of Colorado), is there any reason not to push a 1.0.0 release?
I think that we can move forward with release asap. If we find any bugs, we will fix them.
I have no issue with a release @kreynen .
One thing to fix before releasing simplesamlphp_auth: it seems like the current Backdrop version is based on 7-x.2.0-alhpa2
(please confirm). We usually want to keep a matching major version number when we port to Backdrop, so the correct version would be 1.x-2.0.0-alpha1
And the correct dev version should be 1.x-2.x
rather than 1.x-1.x
@argiepiano correct, started from 7.x-2.0-alpha2
Thanks. Then it'd be a good idea to rename the dev branch before releasing
i see. @kreynen any issue with me doing that now? I don't think there are outstanding PRs
This all makes sense to me. It lets @irinaz move forward, doesn't block/disrupt any other work and gives any future users a better sense of where the code came from and the current state.
@joelsteidl feel free to create the 1.x-2.0 branch, git tag 1.x-2.0.0-alpha2
and push that.
Well continue working on cleaning this up and moving towards a 2.0.0 release after that
Perfect - thank you so much! I will uninstall current version and install released version when it is ready. @joelsteidl , @kreynen @argiepiano - thanks a lot!
@kreynen I renamed the 1.x-1.x
branch as 1.x.2.x
and tagged a release https://github.com/backdrop-contrib/simplesamlphp_auth/tags
I see a tag but not a release. You may want to go through with a release. Doing a release has the benefit that Backdrop's Installer will be able to find the module through the Backdrop module portal. Backdrop will package the module with the version, which makes it very convenient when there is a new release - your site will indicate that a new release is ready.
As it is right now, the module has not been packaged, and it's not appearing in the Backdrop list of modules.
https://github.com/backdrop-ops/contrib#releases makes is sound like you only need to push a tag.
2. Push the tag up to the contrib repository.
3. Visit the page for the tag in GitHub, and create a release from that page.
4. Wait about 5 seconds, then refresh the GitHub release page.
Is that not accurate? Does someone need to follow the steps in the normal GitHub release process?
All is good now. I didn't see the release when I first checked. Now it's there, and has been picked up by backdropcms.org
Great. Showing up on https://backdropcms.org/modules now too. @joelsteidl did you have to do anything else?
Is the issue that it works, but takes longer than 5 seconds now?
@argiepiano @kreynen I created a release.
@kreynen , should we mark this issue closed? thanks you, @kreynen @joelsteidl @argiepiano !!!
I merged a bunch of bug fixes today. @kreynen @argiepiano @irinaz, do you care to test before I make a next release? (I think I already have most of these patches running on the sites in question but I'm also going to test the latest on a few dev sites of my own.)
I mentioned https://github.com/simplesamlphp/simplesamlphp/issues/1689 and this module as a potential solution for the Symfony 6 compatibility issues. We're working on the scope of work to migrate a few CU sites to Backdrop.