Skip to content

Chore: Fix Playground loading issue, simplify release workflow, fix ESLint test parsing, and Improve test docs#35

Open
danish17 wants to merge 5 commits intodevelopfrom
fix/playground
Open

Chore: Fix Playground loading issue, simplify release workflow, fix ESLint test parsing, and Improve test docs#35
danish17 wants to merge 5 commits intodevelopfrom
fix/playground

Conversation

@danish17
Copy link
Collaborator

This PR improves the project's release reliability, streamlines CI/CD, and resolves development environment issues.

Key Changes

  • WordPress Playground: Fixed an issue where WordPress Playground was unable to load the plugin reliably from the statically.io CDN. The plugin source has been switched to use GitHub Releases for better stability.
  • Release Workflow: Simplified the GitHub Actions release process by removing the automated deployment to the dist branch. The workflow now focuses on creating GitHub releases and uploading the built plugin ZIP.
  • ESLint Configuration: Resolved parsing errors in tests/js/setup.ts and other test files by including tests/js/tsconfig.json in the ESLint parserOptions.project configuration.
  • Test Documentation: Refined JSDoc comments in view.test.ts to include explicit parameter and return types for helper functions, improving developer experience and code clarity.

Verification

  • Plugin now loads reliably in WordPress Playground via GitHub Release URL.
  • ESLint parsing errors in the test suite are resolved.
  • GitHub Actions workflow is updated and valid.
  • Unit tests pass successfully.

@danish17 danish17 self-assigned this Feb 12, 2026
@theMasudRana
Copy link
Collaborator

Update: I tested this blueprint from this branch, and it worked 100% of the time with very fast load times. https://github.com/rtCamp/carousel-kit/blob/fix/playground/blueprint.json

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