-
-
Notifications
You must be signed in to change notification settings - Fork 618
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
Releases? #1631
Comments
Mainly because I'm very busy and publishing is currently not fully automatic. I'm going to try to find some time and configure release-please or something similar over the weekend |
Re help - any release workflow automation would be appreciated. Also one blocker is coverage check that is failing on PRS where source branch outside of this repo. Need to fix the issue or remove the check |
Hey @sidorares, I appreciate how difficult these things are when they're not automated. I believe @daniellockyer fixed a similar issue with PR coverage checks on our repos so hopefully he can take a look at that next week. He's also a dab hand at workflow automation and I'm sure he can help more. |
Hey @sidorares 👋🏻 Apologies for stepping in much later. I'd be more than happy to help given we also fixed up the node-sqlite3 release process too 🙂 I see you've pushed a commit for the coverage check permissions. Shall I take a look into the release-please GHA? |
Thanks @daniellockyer! yes, I was thinking to add release-please I'll describe current manual release process and we can discuss what automated would look like:
Sometimes where I feel changes are too big there is an I still prefer manual changelog, my ideal workflow would be this:
Does not have to be that fancy though. |
@sidorares This should all be doable AFAICT 🙂 My only question is that release-please uses conventional commits to determine the bump type. This repo doesn't seem to use that right now, so the suggested version might be off. Is that something you'd be open to using? |
refs sidorares#1631 - this workflow integrates Google's `release-please-action`, which seems like the best option for using release-please and GHA - if a release was created, it'll also automatically publish to npm
In the mean time, I've opened #1647 🙂 |
@sidorares Is there anything else I can do to help with a release? 🙂 |
hey @daniellockyer looks like release-please action is working and doing its job, I just need to check what I want to do with the first release. Looks like if I merge #1648 it'll publish v2.3.4 do you know how do I force major release? Should I push Sorry for the silly questions, the docs seems to answer them but don't want to do anything dumb acciedentally and then rush to fix that |
I believe you can do either that, or temporarily edit the workflow to have |
@sidorares is there any chance of the release getting published before the end of the month? |
Great to see this collaboration from @ErisDS and @daniellockyer from Ghost here. I'd love to see a release for this as well since I'm depending in MySQL for Azure running as the backend for my blog and I've been waiting for this fix for some time now. I'm going to throw in a sponsorship for @sidorares as well. I have a small taste of how hard and time-consuming maintaining an OSS project can be especially when you have other obligations as well, so every little bit helps :) Looking forward to seeing the next release! |
Thanks for your support @stebet @silverbullettruck2001 was going to write "yes" but just realised its already Nov 1st. I really want to have a test release-please driven release first, I'll prepare a manual changelog and push "release-as: 3.0.0-rc0" some time later tomorrow. Ifter that hopefully going to be just a matter of merging a PR generated by release-please action moving forward Sorry for keeping delaying this, unfortunately lots of other commitments preventing for fully focusing on the work here. Thanks again everyone who helped |
@sidorares That is great news! Thanks again for you continued dedication to this. I definitely understand why you are prioritizing |
Any update on release? Thanks. |
@ErisDS @aysark @silverbullettruck2001 @daniellockyer for some reason publish step failed in the workflow, had to publish manually: https://github.com/sidorares/node-mysql2/actions/runs/3404535753/jobs/5661839917 |
Sorry to complain, but I believe that a release candidate "v3.0.0-rc.1" should not be noted as latest on npm. |
@sidorares Thank you so much 🎉 just a headsup @daniellockyer is taking a well-earned screen break this week, but I'll get him to help with this asap next week. @rafipiccolo that's how npm works by default. Although it can be changed later using |
ok, i was only reporting. i'll adapt my cicd in any case. |
good point @rafipiccolo , you are right changed v2.3.3 tag back to
|
no rush @ErisDS, I can continue to publish manually, it's already easier from now on. We'll get everything running smoothly eventually |
TIL ( I knew about tags but now I know how to set them at publish time and how to change later ) |
Sorry for being defensive, I just know how it feels when you finally ship the thing everyone asks for & the replies are all "And another thing..." 😬 Really appreciate you getting this out. I know @daniellockyer is gonna be stoked, and there's a tonne of people in the Ghost community who are gonna be over the moon to be able to upgrade as well ❤️ |
@sidorares Congrats! 🎉 Thank you SOOO much for getting this out! I will be interested to see you get the automation completed for the next release so that this process can be much easier for you! 👍 😃 |
@sidorares I am sorry to report that I think there is something odd going on with the RC that was generated. When I do an |
@silverbullettruck2001 I just installed the rc and I'm getting the right types for it. Looking at https://unpkg.com/browse/[email protected]/index.d.ts and that looks right as well. Are you pulling in a types file from somewhere else or an old version or something? |
@MasterOdin I was able to fix this so that I am getting the complete types now. However, I am having issues getting the cleartext part to work. Here is an example of what I am doing. Is this how it should be coded to specify the cleartext plugin? When I use this I am still getting
|
On that note, when can we expect Ghost to update this dependency @ErisDS @daniellockyer ? |
Ping :) |
Ghost's dependency updates are largely automated, but won't pick up the RC. We will look at doing a manual update in the new year if there's no movement on an official v3 & it seems the RC is stable 🙂 |
@daniellockyer I think the problem was on my side, I incorrectly configured npm token, though error messages were not very helpful in debugging that. Looks like after few forced release-please runs its all working OK now and we should see a release pushed to npm after every merged PR Not sure how to deal with dependabot PRs, I'd rather batch them to avoid noise in releses. Do I just close automatic release-please PR? Or keep it open while I merge PRs and only merge release PR when I'm ready to push? I guess I'll try that next time Closing this for now, thanks everyone who helped with actions code, advises, friendly pings and any other support 🤟 |
@sidorares Apologies, I missed the earlier ping! This is great - thanks for getting the release out 🙂 |
It's really great to see so many commits + releases in this project again 😀 |
Hi there,
My team and I (Ghost) are trying to understand why this repo doesn't get releases?
There are regular updates to the repo but noone is benefitting from the changes - the last release was back in November 2021. At which point the releases had been every couple of weeks.
We have users specifically waiting on #1438, which was merged 10days after the last release, now almost a year ago.
Is there a problem with the release process? Something we could possibly help with? cc @daniellockyer
We really appreciate all of the hard work going on here - we would love to make use of it!
The text was updated successfully, but these errors were encountered: