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

feat(lune): add optional farm build to lune #33

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

twnlink
Copy link
Member

@twnlink twnlink commented Jul 31, 2024

This pull request introduces optional Farm support for Vite / Rollup plugin compatibility.

Building with Farm is currently reliant on esbuild for the final build step unfortunately due to the way vite-plugin-solid works.

Farm support does not include SCSS out of the box due to being unable to get the official Farm Rust plugin working, but it should be doable via a Vite / Rollup plugin.

Ideally we could eventually resolve both of these issues and remove the dependency on esbuild, which should theoretically make plugin building faster.

Copy link
Member

@yellowsink yellowsink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems okay apart from the postprocessing. just those questions i have also.

"readline/promises",
],
platform: "node",
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to bundle the solid compiler due to npm's inability to pin deps-of-deps so this change is a no-go

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fix it then lol

packages/lune/src/config.ts Show resolved Hide resolved
packages/lune/tsconfig.json Show resolved Hide resolved
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