Skip to content

include .jsx files in esbuild pipeline to fixes #899 (broken options)#900

Merged
robruiz merged 1 commit intowprig:developfrom
erdmann040:fix/issue-#899
Sep 5, 2025
Merged

include .jsx files in esbuild pipeline to fixes #899 (broken options)#900
robruiz merged 1 commit intowprig:developfrom
erdmann040:fix/issue-#899

Conversation

@erdmann040
Copy link
Contributor

…scripts (fixes #899)

Description

Addresses issue #899

When the admin script source was renamed to .jsx, the build no longer produced index.min.js and threfor the options compontent did not work anymore.

Update eslint.config.js with a minimal override to ensure build and configuration scripts are linted and may import from devDependencies. All .js files are now linted unless they are listed in the ignore section.

List of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • This pull request relates to a ticket.
  • This code is tested.
  • [-] This change has been added to CHANGELOG.md
  • I want my code added to WP Rig.

@erdmann040 erdmann040 requested a review from robruiz as a code owner September 4, 2025 19:57
@erdmann040 erdmann040 force-pushed the fix/issue-#899 branch 3 times, most recently from be9d357 to 282fd6e Compare September 4, 2025 23:01
@erdmann040 erdmann040 changed the title Build: include .jsx in esbuild pipeline (fixes #899); smaller changes to eslint.config.js … Build: include .jsx in esbuild pipeline (fixes #899) Sep 4, 2025
@erdmann040 erdmann040 changed the title Build: include .jsx in esbuild pipeline (fixes #899) include .jsx files in esbuild pipeline to fixes #899 (broken options) Sep 4, 2025
@robruiz robruiz merged commit c57e3b5 into wprig:develop Sep 5, 2025
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