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

Next release steps #395

Closed
7 tasks done
eddelbuettel opened this issue Sep 2, 2024 · 7 comments
Closed
7 tasks done

Next release steps #395

eddelbuettel opened this issue Sep 2, 2024 · 7 comments
Assignees

Comments

@eddelbuettel
Copy link
Member

eddelbuettel commented Sep 2, 2024

The following checklist may help, we can expand / add entries as needed

  • merge PR #394 to correct bds()
  • rebase branch feature/release_3.24.6.1 following the merge
  • merge PR #4 in repo blp to provide current libraries (see tests passed in issue #393), it also contains a prior small PR #5
  • merge PR #396, then rebase branch feature/release_3.24.6.1
  • create PR off branch feature/release_3.24.6.1 and merge
  • encourage testing of the merged branch
  • roll version number etc and submit to CRAN

We could also switch to 'squash and merge' for a more linear history, I do that now in a few other releases. Not too important though.

We could also switch to 'person A creates PR, person B approves, person A merges' which is more common. Not too important though.

@johnlaing
Copy link
Contributor

Approved and merged #394 . I rather like the process where the approver merges because it embeds the workflow in the git history itself, rather than just an artifact of the GH PR. Not a strongly held view, though.

Would you like to do the rebase for the release branch? I also have a change to submit there to fix/enhance Makevars.win. Once rebased I can push that up and we can have a look.

@eddelbuettel
Copy link
Member Author

eddelbuettel commented Sep 5, 2024

I rather like the process where the approver merges because it embeds the workflow in the git history itself.

It is strictly the same. But not 'squash-merging' we get this no matter whether it is you or me merging:

image

But it would put my name at the top-right for the last commit. That is how every other GH repo I work on with various people does it, work included: 'developer' prepares PR, 'reviewer' looks it over, may demand changes, eventually approves and then 'developer' merges the branch he or she created when he or she deems appropriate.

But we can do it differently here too if you insist. It does not really matter given the amount of changes we have here.

Would you like to do the rebase for the release branch?

Yes, done.

@johnlaing
Copy link
Contributor

I'm comfortable that the new libraries work, so those are merged.

There's one more extra Name call that I found in bsrch. Relatedly, I figured out how to use the global example searches so we can have an example (and by extension, test case) that actually works. I'll submit a patch for both of those.

@eddelbuettel
Copy link
Member Author

eddelbuettel commented Sep 13, 2024

Made an announcement to r-sig-finance just in case, we can let it sit and hopefully be tested for a few days.

And then release as 0.3.15 to CRAN.

@mtkerbeR
Copy link
Contributor

Thank you very much for your work on the release.
I did a few more tests using the windows binary builds linked in the announcement ... for me, everything worked as expected. 👍

@eddelbuettel
Copy link
Member Author

Planning to release to CRAN on Wednesday.

@eddelbuettel
Copy link
Member Author

Passed without issues in 'autoprocessing'. Nice.

image

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