-
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
Improve reporting to users #246
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:
🚫 6 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:
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:
🚫 71 errors
This bot provides automated PHP linting and PHPCS scanning. For more information about the bot and available customizations, see our documentation.
Posting will continue in further review(s)
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.
Previous scan continued.
…to vipgoci_report_submit_pr_generic_support_comment()
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:
🚫 27 errors
This bot provides automated PHP linting and PHPCS scanning. For more information about the bot and available customizations, see our documentation.
Posting will continue in further review(s)
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.
Previous scan continued.
Improve reporting to users in pull request feedback and in the command line.
TODO:
--version
option.--help
in a better way..vipgoci_options
[ In GitHub reviews, report which options were altered #145 ]PHPDoc
comments for new or updated functions (main code only; for VIP)vipgoci_github_wait()
to two seconds to avoid GitHub API limit errors.vipgoci_github_pr_review_submit()
should be renamed and moved toreports.php
. [ Move functions and functionality between files #142 ]vipgoci_github_pr_generic_comment_submit_results()
should be renamed and moved toreports.php
. [ Move functions and functionality between files #142 ]vipgoci_github_pr_generic_support_comment_submit()
toreports.php
and rename tovipgoci_report_submit_pr_generic_support_comment()
.vipgoci_github_wait()
togithub-misc.php
vipgoci_option_array_handle()
to cover empty parameter.vipgoci_github_wait()
.vipgoci_util_php_interpreter_get_version()
.vipgoci_output_sanitize_version_number()
.vipgoci_output_html_escape()
.vipgoci_phpcs_get_version()
.vipgoci_report_create_scan_details_list()
.vipgoci_report_create_scan_details_software_versions()
vipgoci_report_create_scan_details_php_lint_options()
vipgoci_report_create_scan_details_phpcs_configuration()
vipgoci_report_create_scan_details_svg_configuration()
vipgoci_report_create_scan_details_auto_approve_configuration()
vipgoci_report_results_to_github_were_submitted()
tests/integration/PhpcsScanPossiblyUseNewStandardFileTest.php
so to check value of$options['phpcs-standard-original']
vipgoci_options_read_repo_file()
updates$options['repo-options-set']
.