Skip to content

Feat/remove engine builder providers #255

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

Conversation

arpitkarnatak
Copy link
Contributor

Motivation

Fixes #251

Note: There are 2 tests failing, but they fail on the latest main too and not due to the moving of these two crates into contender_core

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

@arpitkarnatak arpitkarnatak requested a review from zeroXbrock as a code owner May 25, 2025 12:13
@zeroXbrock
Copy link
Member

zeroXbrock commented May 28, 2025

@arpitkarnatak
Sorry if there are a lot of new conflicts -- #243 was just merged, and made a lot of changes to the bundle provider.
This may be easier to just re-do from a fresh branch, but up to you. Apologies for the inconvenience. Thanks for the PR!

@arpitkarnatak
Copy link
Contributor Author

No worries, closing this one and created a new one on #263

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.

move engine_provider & bundle_provider to core
2 participants