Skip to content

Conversation

@derrabus
Copy link
Member

This PR proposes to perform the version bumps that we've done on https://github.com/symfony/acl-bundle already:

  • Symfony 4.4
  • PHP 7.2

Symfony 3 is EOL and we don't develop features for PHP 7.1 anymore. This would also allow us to drop some compatibility code as well as modernize methods with property type declarations.

@derrabus derrabus force-pushed the improvement/drop-symfony-3 branch from 31d4c70 to 526417d Compare April 28, 2021 12:57
@derrabus
Copy link
Member Author

derrabus commented Apr 28, 2021

Apparently, getReachableRoleNames did not exist on Symfony 4.4 already. I'll revert that part later. Done.

@derrabus derrabus force-pushed the improvement/drop-symfony-3 branch 3 times, most recently from 064476e to 39b4eaf Compare April 28, 2021 18:12
@acrobat
Copy link
Contributor

acrobat commented Apr 28, 2021

Would it be an option to first release a minor with the dbal 2.13 compatibility fixes? And have this PR included in the next minor?

@derrabus derrabus force-pushed the improvement/drop-symfony-3 branch from 39b4eaf to dcf17ad Compare April 28, 2021 18:17
@derrabus
Copy link
Member Author

Would it be an option to first release a minor with the dbal 2.13 compatibility fixes? And have this PR included in the next minor?

I can tag a patch release for the DBAL compatibility, I suppose.

@derrabus
Copy link
Member Author

@acrobat v3.1.2 was tagged.

@acrobat
Copy link
Contributor

acrobat commented Apr 28, 2021

Thanks @derrabus!

@derrabus derrabus merged commit 8bc635b into symfony:main May 17, 2021
@derrabus derrabus deleted the improvement/drop-symfony-3 branch May 17, 2021 12:14
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.

2 participants