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

Getting a deprecation warning about this step in the action in V3 / V4 #8

Closed
cAstraea opened this issue Oct 31, 2022 · 4 comments · Fixed by fabpot/local-php-security-checker#70

Comments

@cAstraea
Copy link

Warning: The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

@jrfnl
Copy link

jrfnl commented Nov 5, 2022

Looks like this is actually related to the underlying tooling: fabpot/local-php-security-checker#67

@fabpot
Copy link
Contributor

fabpot commented Dec 7, 2022

Fixed now

@fabpot fabpot closed this as completed Dec 7, 2022
@jrfnl
Copy link

jrfnl commented Dec 7, 2022

@fabpot Thanks Fabien!

@spaze
Copy link
Contributor

spaze commented Jan 3, 2023

NB: This has not been released yet, it was fixed in the local PHP checker 2.0.6, but the fix wasn't propagated up to this action:

  • The CLI tool requires github.com/fabpot/local-php-security-checker/v2 v2.0.5
  • This repo has a tag but not a v5 release that would use the CLI image

UPDATE: The CLI tool now uses 2.0.6 (symfony-cli/symfony-cli@11b775a) and the action can specify a tag, not a release, although the release would still be nice (#10)

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

Successfully merging a pull request may close this issue.

4 participants