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

Release version 1.3.3/1.4.0 and syncing the branches #155

Closed
jrfnl opened this issue Mar 16, 2024 · 4 comments
Closed

Release version 1.3.3/1.4.0 and syncing the branches #155

jrfnl opened this issue Mar 16, 2024 · 4 comments
Milestone

Comments

@jrfnl
Copy link
Collaborator

jrfnl commented Mar 16, 2024

Setting the scene

A couple of years back, a number of things were proposed for version 2.0 and work was started on that, but there has been little progress in the past year or so.

In practice this means that there is now one commit (#148) in master, which is not in develop (which is why Dependabot is not yet running) and - aside from the v2 specific patches - there are also a number of commits in develop which should be ported to master.

How to move forward

To make merges and releases more straight forward again, I think it would be good to get the branches back in sync and change strategies a little.

There are basically two options:

  1. Keep the branches as they are and cherrypick select commits from develop to master and visa versa.
    The milestone of the cherrypicked commits should be updated from 2.0.0 to 1.3.x Next.
    This will largely keep things as they are now workflow wise, but keeps the mental overhead of the two branches potentially diverging and having to cherrypick and such.
  2. Cherrypick select commits from develop to master and then rebase develop on top of master.
    This should be combined with setting the default branch back to master, rebasing currently open and viable PRs against master if they are not part of the larger changes for 2.0 and don't contain breaking changes (and switching the branch these PRs are pulled against).
    Moving forward, PRs should be pulled against master (with the exception of 2.0 specific PRs) and when PRs are merged, the master branch should be merged into the develop branch, until develop is ready for the 2.0 release.
    Note: This change does mean that people with active forks, will have to reset their develop branch.

I'd advocate for strategy 2, but would like to hear opinions.

Detailed proposal

I'd be happy to get this sorted and to put the work in to make it happen.

Independently of which workflow is used, I believe the following changes should be included in the next release/cherrypicked to master:

I intend to do some test runs in the next few hours to make sure we'll have a passing build on master with the changes as proposed.

I'll also start preparing the changelog for the 1.3.x release (or 1.4.0, I'll have a look what it should be when I prepare the changelog).

@grogy What do you think ?

@jrfnl jrfnl added this to the 1.3.x Next milestone Mar 16, 2024
@jrfnl
Copy link
Collaborator Author

jrfnl commented Mar 16, 2024

FYI: I've re-milestoned the above listed PRs to 1.3.x will be renaming that milestone in a moment to 1.4.0 as the release contains something which can be considered a new feature.

@jrfnl
Copy link
Collaborator Author

jrfnl commented Mar 17, 2024

I've prepared a feature/release-1.4.0 branch containing the cherrypicks as proposed above + a changelog for the 1.4.0 release based on the above PRs/commits being included in the release.

The builds pass for the 1.4.0 release branch, with the exception of PHP 8.4, but this is not due to this package not being compatible, but due to Nette Tester 1.0 not being compatible with PHP 8.4.

Target release date: Tuesday March 19 (providing @grogy responds in time).

@jrfnl
Copy link
Collaborator Author

jrfnl commented Mar 20, 2024

@grogy Would be great to get this out of the door and as PRs can't be merged with an approval, I can't do anything about this without you getting involved. When can we get this done ?

@jrfnl
Copy link
Collaborator Author

jrfnl commented Mar 27, 2024

In the mean time, the 1.4.0 release has gone out 🎉

@grogy and me discussed the branching today in a call and the conclusion was basically that it would probably be best to work towards getting the 2.0 release tagged soonish and to then drop support for the 1.x branch.

@jrfnl jrfnl closed this as completed Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

1 participant