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

MHCG Pull Requests from original project #1

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

Conversation

markheydon
Copy link
Member

No description provided.

Copy link
Collaborator

@mhcg-bot mhcg-bot left a 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:

🚫 9 errors


hashes-api-scanning skipped

Copy link
Collaborator

@mhcg-bot mhcg-bot left a 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:

🚫 4 errors


hashes-api-scanning skipped

@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg mhcg deleted a comment from mhcg-bot May 30, 2023
@mhcg-bot
Copy link
Collaborator

Invalid PHPCS sniff(s) specified in options or options file. Those have been ignored temporarily. Please update the options so that scanning can continue as expected.

  • Option name: --phpcs-sniffs-exclude
  • Invalid sniff(s): WordPress.Files.FileName.NotHyphenatedLowercase, WordPress.Files.FileName.NotHyphenatedLowercase
    (commit-ID: 144e282).

Copy link
Collaborator

@mhcg-bot mhcg-bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code analysis identified issues

action-phpcs-code-review has identified potential problems in this pull request during automated scanning. We recommend reviewing the issues noted and that they are resolved.

phpcs scanning turned up:

🚫 5 errors


Powered by rtCamp's GitHub Actions Library

@@ -0,0 +1,33 @@
<?php
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected vipgoci-phpcs-scan-hxaf1k.php, but found vipgoci-phpcs-scan-HXAF1k.php (WordPress.Files.FileName.NotHyphenatedLowercase).

@@ -0,0 +1,33 @@
<?php
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 Error: Class file names should be based on the class name with "class-" prepended. Expected class-plugin-name-admin-cli.php, but found vipgoci-phpcs-scan-HXAF1k.php (WordPress.Files.FileName.InvalidClassFileName).

@@ -0,0 +1,33 @@
<?php
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 Error: There must be no blank lines before the file comment (Squiz.Commenting.FileComment.SpacingAfterOpen).

@@ -0,0 +1,57 @@
<?php
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 Error: Filenames should be all lowercase with hyphens as word separators. Expected vipgoci-phpcs-scan-irp3yo.php, but found vipgoci-phpcs-scan-iRP3Yo.php (WordPress.Files.FileName.NotHyphenatedLowercase).

@@ -0,0 +1,57 @@
<?php
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 Error: Class file names should be based on the class name with "class-" prepended. Expected class-plugin-name-cron.php, but found vipgoci-phpcs-scan-iRP3Yo.php (WordPress.Files.FileName.InvalidClassFileName).

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.

2 participants