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

report versions of the used tools #288

Closed
staabm opened this issue Sep 10, 2020 · 2 comments · Fixed by #300
Closed

report versions of the used tools #288

staabm opened this issue Sep 10, 2020 · 2 comments · Fixed by #300
Assignees
Labels
enhancement New feature or request

Comments

@staabm
Copy link

staabm commented Sep 10, 2020

Describe the feature
would be great the build-output would show which version of the tools are used.

I guess we are currently running in one of the bugs the recent composer releases had, but were fixed in the subsequent release (we had 2 or 3 releases yesterday).

since I cannot see which version of composer the build is using I cannot confirm my thesis easily.

grafik

Version
using the latest v2 tag

@staabm staabm added the enhancement New feature or request label Sep 10, 2020
@shivammathur
Copy link
Owner

@staabm By default it will always setup the latest release version.

I had not done this as some tools have a startup time and checking the version adds to the CI runtime. I will run some tests again.
Currently, I would recommend adding a step after setup-php step to check the tool version.
For cs2pr it will be 1.3.0 release. It shows the versions as 1.1.1-dev though as it picks $version in the script.

- name: Check versions
  run: |
    composer -V
    cs2pr version

@staabm
Copy link
Author

staabm commented Sep 10, 2020

Thx for the headsup. Just created a cs2pr issue staabm/annotate-pull-request-from-checkstyle#51

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

Successfully merging a pull request may close this issue.

2 participants