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(hydroflow_plus)!: minimize dependencies pulled into trybuild builds #1611

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shadaj
Copy link
Member

@shadaj shadaj commented Dec 14, 2024

We don't need hydroflow_lang in the runtime builds, do some feature flagging to avoid it.

Copy link

cloudflare-workers-and-pages bot commented Dec 14, 2024

Deploying hydroflow with  Cloudflare Pages  Cloudflare Pages

Latest commit: 8a40669
Status: ✅  Deploy successful!
Preview URL: https://3c2bff7d.hydroflow.pages.dev
Branch Preview URL: https://pr1611.hydroflow.pages.dev

View logs

@shadaj shadaj requested a review from MingweiSamuel December 14, 2024 00:42
…ilds

We don't need `hydroflow_lang` in the runtime builds, do some feature flagging to avoid it.
@MingweiSamuel MingweiSamuel changed the title feat(hydroflow_plus): minimize dependencies pulled into trybuild builds feat(hydroflow_plus)!: minimize dependencies pulled into trybuild builds Dec 14, 2024
@MingweiSamuel
Copy link
Member

Hmm so what does HF+ do that doesn't depend on hydroflow_lang?

@shadaj
Copy link
Member Author

shadaj commented Dec 14, 2024

Hmm so what does HF+ do that doesn't depend on hydroflow_lang?

It still has all the logic for building up the IR, just not compiling it to the final Rust code, which we only need when we're doing trybuild or using the old macro API. The trybuild builds still need to depend on HF+ though because we re-export some runtime dependencies.

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