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

Create 2.0.16 release #16885

Closed
schmunk42 opened this issue Nov 13, 2018 · 24 comments
Closed

Create 2.0.16 release #16885

schmunk42 opened this issue Nov 13, 2018 · 24 comments

Comments

@schmunk42
Copy link
Contributor

schmunk42 commented Nov 13, 2018

Latest stable release is from March.
There have been quite some bugfixes.

2.0.16 was scheduled for 3rd quarter

I think it also makes sense to have a 2.x branch.

https://github.com/orgs/yiisoft/projects/14

@samdark
Copy link
Member

samdark commented Nov 13, 2018

There are some critical issues to be fixed. I've already moved some to next milestone but there are still plenty to check. If you have some time to burn, help is really appreciated.

@samdark samdark added this to the 2.0.16 milestone Nov 13, 2018
@schmunk42
Copy link
Contributor Author

Which ones are the critical ones ... "ready for adoption" + "bug"?

@samdark
Copy link
Member

samdark commented Nov 14, 2018

These are triaged ones. Not necessarily critical.

@schmunk42
Copy link
Contributor Author

Can you post a link to a filtered view for triaged issues?

@samdark
Copy link
Member

samdark commented Nov 16, 2018

@schmunk42
Copy link
Contributor Author

I think it would be good to adjust the release policy for 2.x slightly.
When looking at the changelog there are almost 80(!) bugs fixed for 2.0.16 - why keep them back? Issuing a patch release is perfectly fine.

The remaining open ones are looking rather hard and complicated to me (half of them are for PgSQL, MSSQL and Oracle), IMHO there's no need to wait until all of them are fixed to get a new release.

We're having a much greater flexibility regarding version numbers now.

@samdark
Copy link
Member

samdark commented Nov 16, 2018

Issuing a patch release is perfectly fine

Not till we're sure critical stuff is fixed. Non-critical issues should first be moved to next milestone.

@schmunk42
Copy link
Contributor Author

I'd at least at a "critical" label then.

@samdark
Copy link
Member

samdark commented Nov 16, 2018

We haven't triaged all issues for the milestone so haven't applied labels yet.

@machour
Copy link
Member

machour commented Jan 14, 2019

I went carefully over all issues with milestone 2.0.16 and postponed all non-critical issues to 2.0.17.

My definition of "non-critical" being: issue was already present for more than two 2.0.X releases, doesn't look like a 2.0.15 regression and isn't vital to 2.0.16.

We're down to 7 issues, including this one:

@samdark
Copy link
Member

samdark commented Jan 16, 2019

Another thing: #16891. Regression.

@samdark
Copy link
Member

samdark commented Jan 17, 2019

Need to release ElasticSearch before doing core release. See #16891

@samdark
Copy link
Member

samdark commented Jan 18, 2019

@samdark samdark moved this from To do to In progress in 2.0.16 release (February 01, 2019) Jan 20, 2019
@machour
Copy link
Member

machour commented Jan 21, 2019

I've been testing 2.0.16 on my biggest Yii 2.0.15 application for a week.

I just came across a breaking change in the Inflector::slug() method, which I think was introduced in #15204

2.0.15: Inflector::slug('0-100 Km/h', '_'); // 0_100_kmh
2.0.16: Inflector::slug('0-100 Km/h', '_'); // 0100_kmh

Not sure if this is the intended behavior, but I can foresee a LOT of migrations breaking & confused devs.

/cc @bizley

@samdark
Copy link
Member

samdark commented Jan 21, 2019

@machour no issues other than that?

@machour
Copy link
Member

machour commented Jan 21, 2019

So far so good 🏅 aside from this issue (and I'm covering a load of Yii features)

@schmunk42
Copy link
Contributor Author

Also related to 2.0.16 - #17049

@bizley
Copy link
Member

bizley commented Jan 21, 2019

@machour this looks like a mistake - I'll prepare fix.

@SilverFire
Copy link
Member

It seems that we are ready to release?

@machour
Copy link
Member

machour commented Jan 21, 2019

There's still #16780, but not sure it's a regression or if it can be postponed.

I think we need to test the release a bit more with basic/advanced & devs apps, seeing that we just fixed two "nasty" regressions today. Feb 1 seems like a good release date.

@samdark
Copy link
Member

samdark commented Jan 21, 2019

@SilverFire we need to review issues of basic/advanced apps. I'm starting to write announcement meanwhile.

@samdark
Copy link
Member

samdark commented Jan 27, 2019

Seems all the preparation things in this repo are completed. Closing. Will track things in a project: https://github.com/orgs/yiisoft/projects/14

@samdark samdark closed this as completed Jan 27, 2019
@samdark samdark moved this from In progress to Done in 2.0.16 release (February 01, 2019) Jan 27, 2019
@s1lver
Copy link
Member

s1lver commented Jan 28, 2019

Seems all the preparation things in this repo are completed. Closing. Will track things in a project: https://github.com/orgs/yiisoft/projects/14

This link is not available

@samdark
Copy link
Member

samdark commented Jan 28, 2019

Fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

No branches or pull requests

6 participants