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.

Sign up for GitHub

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

Jump to bottom

Convert resolveTailwindConfig.js to ESM #108

Merged
merged 5 commits into from
Feb 26, 2024
Merged

Convert resolveTailwindConfig.js to ESM #108

merged 5 commits into from
Feb 26, 2024

Conversation

ptrckvzn
Copy link
Member

Use dynamic imports instead of require()

@ptrckvzn
Copy link
Member Author

I tested using CommonJS by renaming my tailwind config to tailwind.config.cjs, changing the export to module.exports = {} and updating the blast config tailwind_config_path key. It seems to work well on a basic app.

@ptrckvzn
Clean up resolveTailwindConfig.js
2dc21c0
@ptrckvzn
Remove async statement on parseConfig fn
6508b75
@ptrckvzn
Merge branch '1.x' into feature/esm-support
f8f9437 
* 1.x:
  Bump follow-redirects from 1.14.8 to 1.15.5 (area17#112)
  Bump @babel/traverse from 7.22.8 to 7.23.9 (area17#111)
  Bump ip from 2.0.0 to 2.0.1 (area17#110)
  Bump postcss from 8.4.21 to 8.4.35 (area17#109)
  Fix issue where defining name of root story would render as a child story
  Revert tailwind ESM change
@mrtimbrook mrtimbrook merged commit 8982118 into area17:1.x Feb 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ptrckvzn @mrtimbrook