Skip to content

phpcs:chore - Update PHP_CodeSniffer to show severity and code#935

Merged
nathanmartinszup merged 1 commit into
mainfrom
hotfix/php-codesniffer
Jan 20, 2022
Merged

phpcs:chore - Update PHP_CodeSniffer to show severity and code#935
nathanmartinszup merged 1 commit into
mainfrom
hotfix/php-codesniffer

Conversation

@wiliansilvazup

Copy link
Copy Markdown
Contributor

I update the PHP_CodeSniffer tool to have a little
more information such as:

  • All warnings can be shown as long as their severity is above 5
  • All vulnerabilities pointed out have a correct level
    of severity based on the tool
  • We now demonstrate the vulnerable code in the analysis output

@wiliansilvazup wiliansilvazup changed the title Update PHP_CodeSniffer to show severity and code phpcs:chore - Update PHP_CodeSniffer to show severity and code Jan 19, 2022
@wiliansilvazup
wiliansilvazup force-pushed the hotfix/php-codesniffer branch 2 times, most recently from 4ded218 to 825df6a Compare January 19, 2022 20:21
Comment thread internal/services/formatters/php/phpcs/formatter.go Outdated
Comment thread internal/services/formatters/php/phpcs/formatter.go Outdated
Comment thread internal/services/formatters/php/phpcs/formatter_test.go Outdated
Comment thread internal/services/formatters/php/phpcs/formatter_test.go Outdated
Comment thread internal/services/formatters/php/phpcs/formatter.go Outdated
Comment thread internal/services/formatters/php/phpcs/formatter.go Outdated
I update the PHP_CodeSniffer tool to have a little
more information such as:
- All warnings can be shown as long as their severity is above 5
- All vulnerabilities pointed out have a correct level
of severity based on the tool
- We now demonstrate the vulnerable code in the analysis output

Signed-off-by: wilian <[email protected]>
@nathanmartinszup
nathanmartinszup merged commit 5d8b435 into main Jan 20, 2022
@matheusalcantarazup
matheusalcantarazup deleted the hotfix/php-codesniffer branch January 20, 2022 18:07
@wiliansilvazup wiliansilvazup linked an issue Jan 21, 2022 that may be closed by this pull request
2 tasks
matheusalcantarazup pushed a commit that referenced this pull request Jan 21, 2022
I update the PHP_CodeSniffer tool to have a little
more information such as:
- All warnings can be shown as long as their severity is above 5
- All vulnerabilities pointed out have a correct level
of severity based on the tool
- We now demonstrate the vulnerable code in the analysis output

Signed-off-by: wilian <[email protected]>
(cherry picked from commit 5d8b435)
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 this pull request may close these issues.

PHP_CodeSniffer can be used better

3 participants