Skip to content

Conversation

@aledesma-godaddy
Copy link
Contributor

Description

I deduplicated the Cypress spec matrix to avoid redundant runs and updated failure artifact names to include the WordPress and PHP versions. This makes artifacts unique per environment and preserves multiple failures when they happen across versions.

Screenshots

Before:
Screenshot 2025-08-12 at 2 12 16 PM

Types of changes

Bug fix (non-breaking change which fixes an issue)

How has this been tested?

I ran the workflow on a PR with changed files, confirmed the printed matrix lists each spec once, and verified failure artifacts are saved with wp<version> and php<version> in their names.

Acceptance criteria

  • Unique spec list without duplicates for changed-file runs
  • Failure artifacts named <spec>-wp<WP>-php<PHP>-fail.mp4
  • Multiple failures across matrix combos produce separate artifacts
  • No change to behavior when tests pass

Checklist:

  • My code is tested
  • My code follows accessibility standards
  • My code has proper inline documentation
  • I've included any necessary tests
  • I've included developer documentation
  • I've added proper labels to this pull request

@aledesma-godaddy aledesma-godaddy merged commit 5d3256c into master Aug 13, 2025
52 checks passed
@aledesma-godaddy aledesma-godaddy deleted the fix-cicd-issues branch August 13, 2025 18:01
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.

3 participants