-
Notifications
You must be signed in to change notification settings - Fork 23
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
Apply WP CS for a few files, move functionality between files #247
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
phpcs scanning turned up:
🚫 1 error
This bot provides automated PHP linting and PHPCS scanning. For more information about the bot and available customizations, see our documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
phpcs scanning turned up:
🚫 1 error
This bot provides automated PHP linting and PHPCS scanning. For more information about the bot and available customizations, see our documentation.
Dismissing review as all inline comments are obsolete by now
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
phpcs scanning turned up:
🚫 1 error
This bot provides automated PHP linting and PHPCS scanning. For more information about the bot and available customizations, see our documentation.
…r_duplicate() to use vipgoci_results_ prefix
…r_duplicate() to use vipgoci_results_ prefix
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
phpcs scanning turned up:
🚫 3 errors
This bot provides automated PHP linting and PHPCS scanning. For more information about the bot and available customizations, see our documentation.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
phpcs scanning turned up:
🚫 5 errors
This bot provides automated PHP linting and PHPCS scanning. For more information about the bot and available customizations, see our documentation.
Dismissing review as all inline comments are obsolete by now
Apply WordPress coding standard for a few files, along with strict types and setting variable types (see #157).
Apply WordPress coding standards, strict types and set variable types for the following files:
auto-approval.php
ap-file-types.php
ap-hashes-api.php
ap-nonfunctional-changes.php
ap-svg-files.php
defines.php
github-commit-status.php
latest-release.php
test.bootstrap.php
results.php
skip-file.php
statistics.php
support-level-label.php
Move functions between files (see [ #142 ]) :
vipgoci_log()
tolog.php
vipgoci_sysexit()
tolog.php
vipgoci_results_sort_by_severity()
toresults.php
vipgoci_cache()
tocache.php
vipgoci_cached_indication_str()
tocache.php
vipgoci_patch_changed_lines()
togithub-misc.php
vipgoci_github_pr_remove_drafts()
togithub-misc.php
vipgoci_blame_filter_commits()
togithub-misc.php
vipgoci_markdown_comment_add_pagebreak()
togithub-misc.php
vipgoci_github_transform_to_emojis()
togithub-misc.php
vipgoci_issues_filter_irrellevant()
toresults.php
vipgoci_issues_filter_duplicate()
toresults.php
vipgoci_github_comment_match()
toresults.php
Apply WordPress coding standards, etc for the new files:
cache.php
github-misc.php
log.php