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

fix: write envConfig template in generateBundle hook #21

Merged
merged 1 commit into from
Mar 31, 2022

Conversation

lukashass
Copy link
Member

This uses the generateBundle hook to emit env-config.template.js the way rollup intends to add files to the bundle, instead of manually writing a file after the bundle is already written to disk (which was an ugly hack).

@github-actions
Copy link

Coverage report

Total coverage

Status Category Percentage Covered / Total
🟢 Statements 100% 29/29
🟢 Branches 90% 18/20
🟢 Functions 100% 8/8
🟢 Lines 100% 43/43

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Report generated by 🧪jest coverage report action from 463df1e

@lukashass lukashass requested a review from anbraten March 31, 2022 21:01
@lukashass lukashass enabled auto-merge (squash) March 31, 2022 21:35
@lukashass lukashass merged commit 5b07ca2 into main Mar 31, 2022
@lukashass lukashass deleted the use-generate-bundle-hook branch March 31, 2022 22:16
@github-actions
Copy link

🎉 This PR is included in version 4.0.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants