Skip to content

Fix GitHub Actions build and plugin checks (closes #40)#49

Open
9jaDevo wants to merge 3 commits intomarinsagovac:masterfrom
9jaDevo:master
Open

Fix GitHub Actions build and plugin checks (closes #40)#49
9jaDevo wants to merge 3 commits intomarinsagovac:masterfrom
9jaDevo:master

Conversation

@9jaDevo
Copy link
Contributor

@9jaDevo 9jaDevo commented May 10, 2025

What this does

  • Adjusts the GitHub Actions workflow (.github/workflows/wordpress-plugin-check.yml) to:
    • Exclude hidden files and folders from the plugin check
    • Ensure rsync/prepare() usage to satisfy PHPCS
  • Cleans up the SQL calls in:
    • classes/admin/class-paywaydata-list-table.php
    • classes/class-wc-tpayway.php
      so that all queries use $wpdb->prepare() correctly and caching
  • Adds missing translators: comments and escaping where flagged
  • Bumps the “Tested up to” header in the plugin main file to 6.8

Why this is needed

The build has been failing in CI (GitHub Actions) due to:

  • PHPCS errors around direct DB queries and missing placeholders
  • Hidden files being processed in the plugin check
  • Outdated “Tested up to” header filtering the plugin in searches

This PR brings the repo back into a green build state and full compliance with WordPress.org requirements.


Status: Open
Related issue: #40 (opened Dec 1, 2024)

@9jaDevo
Copy link
Contributor Author

9jaDevo commented May 10, 2025

@marinsagovac

@9jaDevo
Copy link
Contributor Author

9jaDevo commented Jul 19, 2025

@marinsagovac Have you looked into these.

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.

1 participant