Skip to content

Internal: Cherry pick [TMZ-1040]#656

Merged
hein-obox merged 57 commits into
3.4from
internal/TMZ-1040-cherry-picked
May 27, 2026
Merged

Internal: Cherry pick [TMZ-1040]#656
hein-obox merged 57 commits into
3.4from
internal/TMZ-1040-cherry-picked

Conversation

@hein-obox
Copy link
Copy Markdown
Member

@hein-obox hein-obox commented May 27, 2026

✨ PR Description

1. Problem & Context

Cherry-pick from TMZ-1040 updating WordPress compatibility, refactoring linting infrastructure, and improving CI/CD test matrix configuration.

2. What Changed (Where)

  • style.css: WordPress "Tested up to" bumped from 6.8 → 7.0
  • composer.json: Renamed lint scripts from lint/lint:fix to lint:php/lint:php:fix
  • package.json: Added comprehensive lint orchestration (CSS, JS, PHP) with individual fix commands
  • .github/workflows/daily-test-matrix.yml: Replaced ternary operator with explicit bash conditional for RUN_TARGETED_TESTS
  • .buildignore: Added .stylelintrc exclusion
  • .stylelintrc: New config file with WordPress stylelint defaults, SCSS plugin, and standard ignores

3. How It Works

Unified linting interface: npm run lint chains CSS/JS/PHP checks; individual commands (lint:php, lint:css, lint:js) delegate to composer or wp-scripts. Workflow fix clarifies boolean logic by explicit if-block instead of ternary, improving readability and preventing edge-case evaluation bugs.

4. Risks

Minor: Bash conditional on line 103 assumes GitHub Actions context provides inputs.run_targeted_tests; malformed input could cause silent failures. Mitigation: test with missing/invalid inputs. CSS linting now active—may surface existing style violations in CI; stage enforcement gradually if needed.

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

hein-obox and others added 30 commits September 5, 2025 11:48
hein-obox and others added 26 commits January 20, 2026 10:44
Co-authored-by: Manor Hazaz <manorhazaz@gmail.com>
* Internal: Bump version [TMZ-1012]

* wip
* Internal: Fix deploy action [TMZ-1013]

* add validation
@hein-obox hein-obox merged commit 602218c into 3.4 May 27, 2026
38 checks passed
@hein-obox hein-obox deleted the internal/TMZ-1040-cherry-picked branch May 27, 2026 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants