Skip to content

Conversation

gtokman
Copy link
Contributor

@gtokman gtokman commented Apr 29, 2025

Copy link
Collaborator

@davidkoski davidkoski left a comment

Choose a reason for hiding this comment

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

We still need the imports in the ContentViews (applications)

@gtokman
Copy link
Contributor Author

gtokman commented Apr 30, 2025

@davidkoski responded to feedback! Agreed on your point that it's used in the test app not library. I didn't see the applications schemes in the project, let me know if there needs to be a follow up PR to add AsyncAlgorithms to those projects.

Copy link
Collaborator

@davidkoski davidkoski left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@davidkoski davidkoski merged commit eb1733c into ml-explore:main Apr 30, 2025
1 check passed
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