Modernize build system and CI/CD - #7
Merged
Merged
Conversation
- Replace gulp with @wordpress/scripts for linting (eslint, stylelint) - Use @kunoichi/grab-deps for JS/CSS build - Use sass + postcss for SCSS compilation - Add wp-dependencies.json auto-registration in hamelp.php - Refactor SearchBoxShortCode to use auto-registered assets - Update JS to ESLint-compliant code with defer strategy - Remove gulpfile.js, .jshintrc - Add .stylelintrc.json Co-authored-by: Claude <noreply@anthropic.com>
- Migrate PHP requirement from 5.4 to 7.4 - Update WordPress compatibility to 6.6+ - Update composer dependencies (PHPCS 3.x, PHPUnit 9.x) - Rename phpcs.xml.dist to phpcs.ruleset.xml with updated rules - Fix PHPCS errors: doc comments, strict comparisons, escaping - Update bin/build.sh for modern deployment Co-authored-by: Claude <noreply@anthropic.com>
- test.yml: Run PHP/JS/CSS tests on PR and push - release-drafter.yml: Auto-draft releases on master merge - wordpress.yml: Deploy to WordPress.org on release publish - wp-outdated.yml: Monthly WordPress version check Co-authored-by: Claude <noreply@anthropic.com>
- Run npm run lint:js for JS files - Run npm run lint:css for SCSS files - Run composer lint for PHP files Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Changes
Build System
CI/CD
test.yml: Run PHP/JS/CSS tests on PR and pushrelease-drafter.yml: Auto-draft releases on master mergewordpress.yml: Deploy to WordPress.org on release publishwp-outdated.yml: Monthly WordPress version checkCode Quality
Test plan
npm run packagebuilds assets correctlycomposer lintpasses🤖 Generated with Claude Code