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.

Sign up for GitHub

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

Jump to bottom

Replace checked-in ml-commons ZIP with a dynamic dependency #495

Closed
Tracked by #1750
jackiehanyang opened this issue Mar 15, 2022 · 2 comments
Closed
Tracked by #1750

Replace checked-in ml-commons ZIP with a dynamic dependency #495

jackiehanyang opened this issue Mar 15, 2022 · 2 comments
Labels
bug Something isn't working v2.0.0

Comments

@jackiehanyang
Copy link
Contributor

Describe the bug
This PR introduced a checked-in ml-commons zip file as dependency used for doc-test. Need to replace this with a dynamic dependency. See more in main issue here.
@jackiehanyang jackiehanyang added Beta bug Something isn't working untriaged labels Mar 15, 2022
This was referenced Mar 15, 2022
Merged
Closed
@jackiehanyang jackiehanyang mentioned this issue Apr 4, 2022
Merged
6 tasks
@jackiehanyang jackiehanyang mentioned this issue Apr 15, 2022
Merged
6 tasks
@dblock dblock mentioned this issue Apr 18, 2022
Closed
14 tasks
@dblock
Copy link
Member

dblock commented Apr 18, 2022

Reopening until #563 is merged.

@dblock dblock reopened this Apr 18, 2022
@vamsi-amazon
Copy link
Member

Closing since 563 is merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working v2.0.0
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@dblock @penghuo @jackiehanyang @vamsi-amazon