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

[make:*] upgrade bundled PHP-CS-Fixer to v3.49.0 #1429

Merged
merged 3 commits into from
Feb 7, 2024

Conversation

jrushlow
Copy link
Collaborator

@jrushlow jrushlow commented Feb 1, 2024

  • adds a sanity check (TemplateLinterTest::testPhpCsFixerVersion) to ensure php-cs-fixer -V matches the expected TemplateLinter::BUNDLED_PHP_CS_FIXER_VERSION const that must match the actual binary file name. (php-cs-fixer-v.9.9.9.phar

@jrushlow jrushlow added the Status: Needs Work Additional work is needed label Feb 1, 2024
@jrushlow jrushlow force-pushed the feature/upgrade-php-cs-fixer branch 2 times, most recently from 9712fe7 to 51e2902 Compare February 7, 2024 10:29
@jrushlow jrushlow changed the title WIP - [make:*] upgrade bundled PHP-CS-Fixer to v3.48.0 [make:*] upgrade bundled PHP-CS-Fixer to v3.49.0 Feb 7, 2024
@jrushlow jrushlow added Feature New Feature and removed Status: Needs Work Additional work is needed labels Feb 7, 2024
@jrushlow jrushlow marked this pull request as ready for review February 7, 2024 10:30
@jrushlow jrushlow merged commit c1aa494 into symfony:main Feb 7, 2024
6 checks passed
@jrushlow jrushlow deleted the feature/upgrade-php-cs-fixer branch February 7, 2024 13:59
@jrushlow jrushlow mentioned this pull request Feb 20, 2024
public function testPhpCsFixerVersion(): void
{
if (str_contains(strtolower(\PHP_OS), 'win')) {
$this->markTestSkipped('Test only runs on linux.');
Copy link
Contributor

Choose a reason for hiding this comment

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

this statement is incorrect, for OSX PHP_OS = 'Darwin' which also contains 'win'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants