Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rollback commonjs plugin in extensions-sdk to fix minified builds #17539

Merged
merged 1 commit into from Feb 16, 2023

Conversation

freekrai
Copy link
Contributor

@freekrai freekrai commented Feb 15, 2023

Description

The rollup commonjs plugin breaks builds when minified (must be a combination issue with the terser as it works when not minified) from version 23.0.5 (pull request rollup/plugins#1363).

Therefore we rollback to version 23.0.4 for now, where minified builds work without any problems.

Fixes #17529

Type of Change

  • Bugfix
  • Improvement
  • New Feature
  • Refactor / codestyle updates
  • Other, please describe:

Requirements Checklist

  • New / updated tests are included
  • All tests are passing locally
  • Performed a self-review of the submitted code

If adding a new feature:

  • Documentation was added/updated. PR:

@freekrai freekrai marked this pull request as draft February 15, 2023 19:35
@freekrai freekrai requested review from paescuj, a team, connorwinston and jaads and removed request for a team February 15, 2023 19:35
@paescuj paescuj force-pushed the fix/extensions-sdk-rollup-terser-rollback branch from 37c79ec to 1aa13c9 Compare February 16, 2023 11:09
@paescuj paescuj changed the title rollback sdk to 0.2.1 Rollback commonjs plugin in extensions-sdk to fix minified builds Feb 16, 2023
@paescuj paescuj marked this pull request as ready for review February 16, 2023 11:16
@paescuj paescuj merged commit c208286 into main Feb 16, 2023
@paescuj paescuj deleted the fix/extensions-sdk-rollup-terser-rollback branch February 16, 2023 14:54
@rijkvanzanten rijkvanzanten added this to the Next Release milestone Feb 28, 2023
meditadvisors pushed a commit to ciso360ai/directus-mod that referenced this pull request Mar 3, 2023
Co-authored-by: Pascal Jufer <pascal-jufer@bluewin.ch>
@paescuj paescuj mentioned this pull request Mar 16, 2023
ThinkZ pushed a commit to ThinkZ/directus-view that referenced this pull request Mar 29, 2023
Co-authored-by: Pascal Jufer <pascal-jufer@bluewin.ch>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 2, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

extensions-sdk v9.22.3 breaking change under certain (unknown) conditions
3 participants