Skip to content

Skip phpstan config & agent files when building#131

Open
agibson-godaddy wants to merge 3 commits intomasterfrom
skip-more-files-on-build
Open

Skip phpstan config & agent files when building#131
agibson-godaddy wants to merge 3 commits intomasterfrom
skip-more-files-on-build

Conversation

@agibson-godaddy
Copy link
Copy Markdown
Contributor

@agibson-godaddy agibson-godaddy commented Apr 2, 2026

Summary

This updates the copy task to skip copying over ethe following file types:

  • phpstan.neon config file
  • CLAUDE.md file
  • .agents/ directory

QA

  1. Use this branch of sake
  2. Run sake build
    • Above mentioned file types are excluded

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the build copy task so non-runtime configuration/agent artifacts aren’t included in the build output produced by sake build, keeping distributions cleaner and avoiding accidental shipping of tooling/AI metadata.

Changes:

  • Exclude phpstan.neon from the build copy globs.
  • Exclude CLAUDE.md from the build copy globs.
  • Exclude the .agents/ directory (including contents) from the build copy globs.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@agibson-godaddy
Copy link
Copy Markdown
Contributor Author

I removed phpstan.neon from the "do not copy" list.

WooCommerce's QIT tests can read this file to customize their PHPStan test. bc7e71a

@agibson-godaddy
Copy link
Copy Markdown
Contributor Author

I removed phpstan.neon from the "do not copy" list.

WooCommerce's QIT tests can read this file to customize their PHPStan test. bc7e71a

Reverted this. Long story short is that we won't be relying on the QIT PHPStan test.

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