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

Suggestion: always display the application version in console #95

Closed
jrfnl opened this issue Feb 20, 2022 · 2 comments · Fixed by #168
Closed

Suggestion: always display the application version in console #95

jrfnl opened this issue Feb 20, 2022 · 2 comments · Fixed by #168
Assignees
Milestone

Comments

@jrfnl
Copy link
Collaborator

jrfnl commented Feb 20, 2022

Currently when running the application, the first line of the console output looks like this:

PHP 8.1.2 | 10 parallel jobs

I'd like to suggest including the name and version of this application in this line, like so:

PHP-Parallel-Lint 1.3.1 | PHP 8.1.2 | 10 parallel jobs

Opinions ?

@jrfnl jrfnl self-assigned this Feb 20, 2022
@mfn
Copy link

mfn commented Feb 21, 2022

Makes sense and good behaviour. Might it break scripts parsing the output? IDE integrations or something? … 🤷🏼

@jrfnl
Copy link
Collaborator Author

jrfnl commented Feb 21, 2022

Might it break scripts parsing the output? IDE integrations or something? … 🤷🏼

Good question, though I don't think that will really be an issue.
IDE integrations and such will normally use a different report format, like --checkstyle or --json, not the plain text console output.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants