@ArnaudTarroux Are you still motivated to work on this feature?

@ArnaudTarroux Are you still motivated to work on this feature?

@fabpot Yes I am, I can work on it next week

Thanks, looks good to me. There is a pending question on the issue: how will one set the current version on real apps?

@nicolas-grekas In the serialization context with the keyversion

In the serialization context with the key version

do you think we need a config option to set the defaultversion context?

In the serialization context with the key version

do you think we need a config option to set the defaultversion context?

Yes maybe in some cases it might be useful. I'll implement it 👍

Thanks, looks good to me. There is a pending question on the issue: how will one set the current version on real apps?

@nicolas-grekas thank for you review.
I've made some changes following your advices, includingversion_compare() instead ofComposer/Semver.
And I've added the possibility to configure the default version in the framework config (serializer.default_version).

We've just moved away frommaster as the main branch to use5.x instead. Unfortunately, I cannot reopen the PR and change the target branch to5.x. Can you open a new PR referencing this one to not loose the discussion? Thank you for your understanding and for your help.