-
Notifications
You must be signed in to change notification settings - Fork 347
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
Versioning / BC handling #1143
Comments
I think we should probably support 2.7, and 2.8+. 2.8+ will work for 3.0+ (Hopefully for quite a while) and 2.7 is the latest LTS (aside from the new 2.8 LTS, but it's basically 3.0 anyway). It's not hard to update from 2.3 and if you're unwilling to update from 2.3 you likely won't be switching to bootstrap 4 anyway so I don't see that as a problem. I looked at grid and form stuff pretty quickly on bootstrap 4 and it seems pretty similar, so we might not have to do too much work there. I'm pretty sure 2.3+ is compatible with 2.7 anyway so it should be fine. I also discussed removing a decent amount of features from this such as base templates, LESS / SASS base templates (can't use LESS anymore anyway for bs4) and removing the composer bridge / automatic installer. For some people it makes it a little easier but it's not good for us to be storing 3rd party libraries inside ours. Here's the comment I left a little while ago discussing a lot of this: #1133 (comment) Can anybody point out any big differences from BS3 to 4? |
Yeah, that was my idea behind the SF <= 2.8 branch as that would be basically just the current state. One thing that made me open this issue is the decent amount of BC code that's now in the bundle. It's adding complexity and as a personal opinion just isn't nice to have forever. Which is why I think it'd be a good idea to have a SF 3.0(+) only branch with all that removed. As for BS 4 there's indeed more. E.g. form-horizontal being gone, form-group no longer including row, the new grid tier and the removal of glyphicons are likely have an impact. |
Ok, let's bump this discussion :). It think we should require Symfony 2.7 as a minimum, since older version (and 3.0) are not supported anymore: http://symfony.com/doc/current/contributing/community/releases.html |
I think it would be great to get some more visibility around the versioning strategy, for example, v3.1.2 to v3.2.0 introduced a breaking change for us with the horizontal/layout changes. Is the aim to follow Semantic Versioning? |
It wasn't a breaking change, hence why we didn't change the major version. You can still use horizontal the same way as before. If it isn't working correctly then please let us know what is wrong and we will fix the issue. |
@isometriks I can open a separate issue for it, perhaps? Is the aim to follow Semantic Versioning, though? |
Yes, the aim is to follow Semver. We made a change in a BC way, so we changed the minor version. If |
@isometriks Thank you, I have created #1220 and #1219 |
Symfony just released version 3 and Bootstrap 4 is also on the horizon.
With both we got a couple of deprecations / BC breaking changes which (will) affect this bundle, but still (will) have older versions that are maintained.
So I thought it might be a good time to discuss a solid versioning strategy for the future to support existing combinations as long as they are maintained with possible bugfixes and also support the newer ones.
In the near future I see these possible combinations that maybe should be maintained in own version-branches:
The text was updated successfully, but these errors were encountered: