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

Show runtime on version command #6222

Merged
merged 1 commit into from Jan 12, 2022
Merged

Show runtime on version command #6222

merged 1 commit into from Jan 12, 2022

Conversation

SpacePossum
Copy link
Contributor

Show the PHP runtime on version command (php-cs-fixer -V), simplify code.

@coveralls
Copy link

coveralls commented Jan 4, 2022

Coverage Status

Coverage increased (+0.01%) to 93.111% when pulling 85bf144 on SpacePossum:master_PHP_runtime_in_output into ee92352 on FriendsOfPHP:master.

src/Console/Application.php Outdated Show resolved Hide resolved
src/Console/Application.php Outdated Show resolved Hide resolved
Copy link
Member

@keradus keradus left a comment

Choose a reason for hiding this comment

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

looking good, but this is one of very few PRs that faces the failure on smoke tests. please take a look ;)

@SpacePossum SpacePossum merged commit 6fe4c69 into PHP-CS-Fixer:master Jan 12, 2022
@SpacePossum SpacePossum deleted the master_PHP_runtime_in_output branch January 12, 2022 13:05
SpacePossum added a commit that referenced this pull request Jan 18, 2022
…gitSha (keradus)

This PR was squashed before being merged into the master branch (closes #6237).

Discussion
----------

DX: Application - better display version when displaying gitSha

before the change of this PR, but after #6222 :
```
ker@dus:~/github/PHP-CS-Fixer λ ./php-cs-fixer.phar
PHP CS Fixer 3.5.0 The Creation by Fabien Potencier and Dariusz Ruminski.
PHP runtime: 8.0.13 (333f15e)
```
it's misleading, as 333f15e is gitSha of Fixer, not of PHP runtime

Commits
-------

1b54c3e DX: Application - better display version when displaying gitSha
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants