Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 11, 2025

This PR removes the brainmaestro/composer-git-hooks dependency and its associated configuration from the project, and updates the minimum PHP version requirement to 8.4.

Changes

  • Removed dependency: Removed brainmaestro/composer-git-hooks from require-dev section
  • Removed hooks configuration: Removed the extra.hooks section that contained pre-commit and pre-push hook definitions
  • Cleaned up scripts: Removed all cghooks-related scripts including:
    • post-install-cmd hooks that ran cghooks commands
    • post-update-cmd hooks that ran cghooks commands
    • cghooks script alias
  • Updated PHP requirement: Added PHP 8.4 minimum version requirement to composer.json
  • Updated CI pipeline: Updated GitHub Actions workflow to use PHP 8.4 for both phpcs and phpunit jobs

Why

The composer-git-hooks package provided automatic git hook management through composer, but this functionality is being removed from the project. All the essential development scripts (check-style, fix-style, test, phpstan, psalm) remain available and can be run manually or through other CI/CD workflows.

The PHP version has been updated to 8.4 to align with modern PHP standards and ensure the pipeline tests against the minimum supported version.

Verification

  • composer.json validates successfully
  • ✅ No references to the package remain in the codebase
  • ✅ All other composer scripts continue to work independently
  • ✅ CI pipeline configured to test with PHP 8.4
Original prompt

移除 brainmaestro/composer-git-hooks 和 extra.hooks


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

@overtrue overtrue marked this pull request as ready for review October 11, 2025 09:22
@overtrue
Copy link
Owner

@copilot 流水线 php 最低 8.4

Co-authored-by: overtrue <1472352+overtrue@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove brainmaestro/composer-git-hooks and extra.hooks Remove brainmaestro/composer-git-hooks and extra.hooks configuration Oct 11, 2025
Copilot AI requested a review from overtrue October 11, 2025 09:28
Copilot finished work on behalf of overtrue October 11, 2025 09:28
Co-authored-by: overtrue <1472352+overtrue@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Oct 11, 2025

@copilot 流水线 php 最低 8.4

Updated CI pipeline and composer.json to require PHP 8.4 as minimum version. Commit: 5a624c1

Copilot AI changed the title Remove brainmaestro/composer-git-hooks and extra.hooks configuration Remove brainmaestro/composer-git-hooks and update PHP minimum to 8.4 Oct 11, 2025
Copilot finished work on behalf of overtrue October 11, 2025 09:31
@overtrue overtrue merged commit 54d1fe5 into master Oct 11, 2025
5 of 6 checks passed
@overtrue overtrue deleted the copilot/remove-composer-git-hooks branch October 11, 2025 09:34
Copilot finished work on behalf of overtrue October 11, 2025 09:34
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