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(stageleft): support crates that have no entrypoints #983

Merged
merged 1 commit into from
Dec 19, 2023
Merged

Conversation

shadaj
Copy link
Member

@shadaj shadaj commented Dec 18, 2023

feat(stageleft): support crates that have no entrypoints

Also includes various bugfixes needed for Hydroflow+.


Stack created with Sapling. Best reviewed with ReviewStack.

Copy link

cloudflare-workers-and-pages bot commented Dec 18, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 5d67faa
Status: ✅  Deploy successful!
Preview URL: https://ca6db8e4.hydroflow.pages.dev
Branch Preview URL: https://pr983.hydroflow.pages.dev

View logs

@shadaj shadaj force-pushed the pr983 branch 2 times, most recently from 4c0f7b3 to 6ac38d6 Compare December 19, 2023 00:42
@shadaj shadaj changed the title feat(hydroflow): support building graphs for symmetric clusters in Hydroflow+ feat(stageleft): support crates that have no entrypoints Dec 19, 2023
@shadaj shadaj changed the title feat(stageleft): support crates that have no entrypoints feat(hydroflow): support building graphs for symmetric clusters in Hydroflow+ Dec 19, 2023
@shadaj shadaj marked this pull request as ready for review December 19, 2023 00:43
@shadaj shadaj changed the title feat(hydroflow): support building graphs for symmetric clusters in Hydroflow+ feat(stageleft): support crates that have no entrypoints Dec 19, 2023
@shadaj shadaj changed the title feat(stageleft): support crates that have no entrypoints feat(hydroflow): support building graphs for symmetric clusters in Hydroflow+ Dec 19, 2023
@shadaj shadaj changed the title feat(hydroflow): support building graphs for symmetric clusters in Hydroflow+ feat(stageleft): support crates that have no entrypoints Dec 19, 2023
@shadaj shadaj requested a review from MingweiSamuel December 19, 2023 00:47
Also includes various bugfixes needed for Hydroflow+.
Comment on lines +15 to +22
let underscored_ident = syn::Ident::new(&underscored, Span::call_site());
quote! { #underscored_ident }
} else {
quote! { crate }
}
}
proc_macro_crate::FoundCrate::Name(name) => {
let ident = syn::Ident::new(&name, Span::call_site());
Copy link
Member

Choose a reason for hiding this comment

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

was there before, but presumably these Span::call_site should get swapped for meaningful spans some day ™️?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yeah... basically all the spans involved here are cross-crate spans, so some day ™️ indeed.

@shadaj shadaj merged commit 7108323 into main Dec 19, 2023
31 checks passed
@shadaj shadaj deleted the pr983 branch December 19, 2023 02:58
nickjiang2378 pushed a commit to nickjiang2378/hydroflow that referenced this pull request Jan 25, 2024
…ct#983)

Also includes various bugfixes needed for Hydroflow+.
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