Skip to content

Conversation

@noniagriconomie
Copy link
Contributor

Bump up the version numbers

published through a *time-based model*:

* A new **Symfony patch version** (e.g. 2.8.15, 4.1.7) comes out roughly every
* A new **Symfony patch version** (e.g. 4.4.9, 5.0.9, 5.1.0) comes out roughly every
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While 5.1.0 strictly is a patch version, I wouldn't list it here (it's a minor release)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should we wait 5.1.1 to be released? as this PR target 5.1, i wanted to put a 5.1 patch version

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's use 5.1.1 ... it doesn't exist yet, but it will exist very soon, so it's OK.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@javiereguiluz
Copy link
Member

@noniagriconomie we've merged #13741, so we need to rebase this one. Thanks!

@javiereguiluz javiereguiluz added this to the 5.0 milestone Jun 3, 2020
@noniagriconomie
Copy link
Contributor Author

I will do it soon ok 👍

@noniagriconomie
Copy link
Contributor Author

@javiereguiluz done, i also just fix an url (tell me if you prefer to do it on another PR)

@javiereguiluz javiereguiluz changed the base branch from 5.1 to 5.0 June 4, 2020 13:22
@javiereguiluz
Copy link
Member

Antoine thanks a lot for rebasing this! Thanks for fixing the outdated link too 🙏

@javiereguiluz javiereguiluz merged commit e57d206 into symfony:5.0 Jun 4, 2020
@noniagriconomie noniagriconomie deleted the patch-9 branch June 4, 2020 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants