-
Notifications
You must be signed in to change notification settings - Fork 590
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
Update ImageSpec documentation #5748
Conversation
Signed-off-by: Kevin Su <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #5748 +/- ##
==========================================
+ Coverage 36.21% 36.31% +0.09%
==========================================
Files 1303 1305 +2
Lines 109644 110019 +375
==========================================
+ Hits 39710 39949 +239
- Misses 65810 65914 +104
- Partials 4124 4156 +32
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Kevin Su <[email protected]>
Signed-off-by: Kevin Su <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some changes requested, otherwise LGTM!
Thank you @pingsutw! This is going to be very helpful |
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
Co-authored-by: Nikki Everett <[email protected]> Signed-off-by: Kevin Su <[email protected]>
@neverett thanks for the review. I've updated the PR. Mind taking a look again. |
What changes were proposed in this pull request?
Update doc and add more examples
Preview: https://flyte--5748.org.readthedocs.build/en/5748/user_guide/customizing_dependencies/imagespec.html