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

Allowed usage with newer version of PHP-CS-Fixer #16

Merged
merged 7 commits into from
Sep 1, 2021

Conversation

Steveb-p
Copy link
Contributor

@Steveb-p Steveb-p commented Jul 29, 2021

Question Answer
JIRA issue N/A
Type improvement
Target package version ?
BC breaks yes
Doc needed no

Allows installation with 3.0 of php-cs-fixer.

Since we've removed the deprecations, we're more than ready to mark this package as compatible with 3.0.

To prevent issues with current packages that rely on 1.0 of this package, and can potentially have invalid code style check declarations (some deprecations are tied directly to php-cs-fixer call) I'd suggest making this a 2.0 release.

Checklist:

  • Provided PR description.
  • Tested the solution manually.
  • Checked that target branch is set correctly.
  • Asked for a review (ping @ibexa/engineering).

@Steveb-p Steveb-p requested review from adamwojs, alongosz and a team July 29, 2021 11:50
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@Steveb-p Steveb-p force-pushed the allow-php-cs-fixer-3.0 branch from 602db1b to 535b53a Compare August 31, 2021 07:38
composer.json Outdated Show resolved Hide resolved
@Steveb-p Steveb-p requested review from adamwojs and a team August 31, 2021 11:50
Copy link
Member

@adamwojs adamwojs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 but we need README.md update as well.

@Steveb-p
Copy link
Contributor Author

+1 but we need README.md update as well.

@adamwojs added readme changes. See if they're to your liking.

@Steveb-p Steveb-p requested a review from a team August 31, 2021 11:59
Copy link
Member

@alongosz alongosz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To prevent issues with current packages that rely on 1.0 of this package, and can potentially have invalid code style check declarations (some deprecations are tied directly to php-cs-fixer call) I'd suggest making this a 2.0 release.

You need to change branch alias in composer.json then

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@adamwojs adamwojs merged commit bde33e6 into master Sep 1, 2021
@adamwojs adamwojs deleted the allow-php-cs-fixer-3.0 branch September 1, 2021 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants