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

Symantec versioning containing BC breaking changes between 5.0.0 and 5.0.1 #40

Closed
ryanolee opened this issue Jun 9, 2020 · 1 comment

Comments

@ryanolee
Copy link

ryanolee commented Jun 9, 2020

Problem

Release versions for Patch and Minor fix (https://semver.org/) level changes have been made with major BC breaks for Twig versions

Example

Between v5.0.0 and v5.0.1 support for twig version 2 and 1 was dropped. By the looks of things this should have been a Major change given it contained changes that broke any backwards compact with older versions of symfony.

Possible fixes

Given v5.01 + have already been released it will be difficult to maintain any updates between versions given it is unclear if version 5.x.x should maintain twig 2 compatibility so securitiy updates / new feature can be added in to the mainline while the current v5.1+ should be moved to v6. Or the we would want to look to maintain compatibility with twig 2 in the v4 set of releases an keep v5 as is ?

Any thoughts @nacmartin @gempir 🤔

@deguif
Copy link
Collaborator

deguif commented Oct 1, 2020

@ryanolee support for Twig 2.x starting from 2.4 was reintroduced in master.
I'm closing this one as support for Twig 1.x is not possible while allowing Twig 2.x and 3.x (namespacing).

@deguif deguif closed this as completed Oct 1, 2020
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

2 participants