Skip to content

Conversation

@odolbeau
Copy link
Member

No description provided.

odolbeau pushed a commit to odolbeau/common that referenced this pull request Apr 11, 2020
@odolbeau odolbeau marked this pull request as draft April 13, 2020 19:31
@bocharsky-bw
Copy link
Member

Hey @odolbeau !

Thank you for creating this draft release! Do you we're ready to merge this and tag the package with v4.0.0 or do you still have any work to do on your mind before the 4.0.0 release here?

@odolbeau
Copy link
Member Author

Good question, this refactoring have been put on hold since we started to work on symfony/symfony#38475 with @welcoMattic
I don't remember where I stopped exactly and I don't have enough time to work on this in the coming days / weeks (neither on the symfony PR unfortunately).

Maybe the work I've done here should be reverted in order to keep a clean repository? As the 4.0.0 have not been tagged yet it should be straightforward (except for crazy people using dev-master but they probably are aware of the risk).

What do you think?

@bocharsky-bw
Copy link
Member

@odolbeau Ah, I see, so refactoring is blocked by symfony/symfony#38475

Maybe the work I've done here should be reverted in order to keep a clean repository? As the 4.0.0 have not been tagged yet it should be straightforward (except for crazy people using dev-master but they probably are aware of the risk).

I hate reverting good work, but yeah, I think it would be a very pragmatic solution here I think and it would help to achieve PHP 8 support easier.

Could you help with reverting that work? Or at least tell me what PRs should be reverted and I'd try to revert it myself

@Nyholm
Copy link
Member

Nyholm commented Mar 26, 2021

Since next release will be 4.0, I'll update branch alias in #41

@Nyholm
Copy link
Member

Nyholm commented Mar 26, 2021

Im looking at the diff. Do you mind if I release 3.1.0 before 4.0.0?

Ie, I just revert the two BC breaking changes, then tag 3.1.0, then apply those changes again. Then we dont have to wait for next Symfony release (probably end of May).

@bocharsky-bw
Copy link
Member

I think it would be perfect to speed up PHP 8 support

@Nyholm
Copy link
Member

Nyholm commented Mar 26, 2021

I changed my mind. It is easier to just add a single commit after 3.0.1:
#43

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

Successfully merging this pull request may close these issues.

3 participants