feat(wanda): Build dependencies in topological order#348
feat(wanda): Build dependencies in topological order#348andrew-anyscale merged 1 commit intomainfrom
Conversation
|
Summary of ChangesHello @andrew-anyscale, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request significantly enhances the Highlights
🧠 New Feature in Public Preview: You can now enable Memory to help Gemini Code Assist learn from your team's feedback. This makes future code reviews more consistent and personalized to your project's style. Click here to enable Memory in your admin console. Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request introduces dependency chain builds to wanda by adding a BuildWithDeps function. The changes are well-structured and include corresponding tests for the new functionality. My review focuses on improving code clarity and test robustness. I've suggested a change to avoid mutating function arguments, which is a good practice for maintainability. I've also recommended strengthening a test case to ensure it properly verifies the expected outcome.
56e00c6 to
f319704
Compare
6f660b1 to
c9aea31
Compare
f319704 to
ae47f27
Compare
ae47f27 to
c1c2a3d
Compare
c9aea31 to
c1346cb
Compare
c1346cb to
988af31
Compare
c1c2a3d to
9dc69db
Compare
988af31 to
388c9c1
Compare
9dc69db to
05c0ded
Compare
388c9c1 to
b822440
Compare
05c0ded to
839b086
Compare
839b086 to
308598c
Compare
b822440 to
587684f
Compare
a289e51 to
2327511
Compare
853d062 to
4c062ac
Compare
2327511 to
fdbd39e
Compare
4c062ac to
d437c29
Compare
86b1c2e to
00b3ca3
Compare
d437c29 to
835bb4c
Compare
00b3ca3 to
1a7af24
Compare
835bb4c to
9bd8415
Compare
b0597a4 to
c0d07ae
Compare
9bd8415 to
2f65672
Compare
e20bf2d to
7c9b96c
Compare
3e32233 to
7bd90b7
Compare
7c9b96c to
cd5a547
Compare
8193927 to
eb8860b
Compare
cd5a547 to
987fd29
Compare
eb8860b to
7f955fd
Compare
543dfc4 to
5a582a1
Compare
Extend Build() to automatically build all dependencies before building the root spec. In local mode, specs are discovered by scanning the repo for *.wanda.yaml files and built in topological order. For a slight optimization, in RayCI mode, only the root spec is built (deps built by prior pipeline steps). Topic: wanda-build-deps Relative: wanda-deps Signed-off-by: andrew <andrew@anyscale.com>
35530d6 to
a384fdf
Compare
Extend Build() to automatically build all dependencies before building the root spec. In local mode, specs are discovered by scanning the repo for *.wanda.yaml files and built in topological order. For a slight optimization, in RayCI mode, only the root spec is built (deps built by prior pipeline steps).
Topic: wanda-build-deps
Relative: wanda-deps
Signed-off-by: andrew andrew@anyscale.com