Skip to content

ci: remove Playwright tests from npm-publish workflow#35

Merged
jaysin586 merged 1 commit into
mainfrom
feature-udpated-version
Jan 29, 2026
Merged

ci: remove Playwright tests from npm-publish workflow#35
jaysin586 merged 1 commit into
mainfrom
feature-udpated-version

Conversation

@jaysin586

@jaysin586 jaysin586 commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

Summary

Remove Playwright e2e tests job from the npm-publish workflow since this package doesn't have Playwright tests configured.

Changes

🔄 CI/CD

  • Remove playwright-tests job from npm-publish workflow
  • Update coverage-report job dependencies to remove playwright-tests dependency
  • Update publish-github-packages job dependencies to remove playwright-tests dependency

Commits

  • 409dc50 ci: remove Playwright tests from npm-publish workflow

🤖 Generated with Claude Code

Summary by CodeRabbit

Release Notes

  • Chores
    • Removed end-to-end testing step from the CI/CD pipeline.
    • Updated workflow job dependencies for downstream tasks.

✏️ Tip: You can customize this high-level summary in your review settings.

Remove Playwright e2e tests job from npm-publish workflow as this
package doesn't have Playwright tests configured. Update job
dependencies to skip the removed playwright-tests job.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Jan 29, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

A GitHub Actions workflow file was modified to remove the Playwright test job step group from the CI/CD pipeline and update downstream job dependencies (coverage-report and publish-github-packages) to no longer depend on the removed job.

Changes

Cohort / File(s) Summary
CI/CD Workflow Configuration
.github/workflows/npm-publish.yml
Removed the playwright-tests job and all associated steps. Updated coverage-report to depend on [check-if-merged, build] and publish-github-packages to depend on [check-if-merged, build, coverage-report], eliminating playwright-tests as a dependency for both downstream jobs.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Hop, skip, the tests take flight,
Playwright steps removed from night,
Dependencies rewired with care,
CI/CD flows cleaner, lighter air!
Streamlined workflows, burrow-bright! 🌿


Comment @coderabbitai help to get the list of available commands and usage tips.

@jaysin586 jaysin586 merged commit eb09886 into main Jan 29, 2026
3 of 4 checks passed
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.

1 participant