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.

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

maint: Move solver::libsolv::parameters to solver::parameters #3378

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

jjerphan
Copy link
Member

@jjerphan jjerphan commented Aug 1, 2024

No description provided.

@jjerphan jjerphan force-pushed the maint/move-libsolv-parameters-up branch from 75d466c to b7de98f Compare August 1, 2024 07:53
Copy link
Member Author

@jjerphan jjerphan left a comment

Choose a reason for hiding this comment

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

A few comments and hints for reviewers.

Copy link
Member Author

Choose a reason for hiding this comment

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

New lines come from libmambapy/src/libmambapy/bindings/solver_libsolv.cpp with no modifications.

Comment on lines +15 to +19
_WARN_TEMPLATE_MSG = (
"`libmambapy.bindings.solver.libsolv.{name}` has been moved "
"to `libmambapy.bindings.solver.{name}` in 2.0. "
"This import path will be removed in 2.2"
)
Copy link
Member Author

@jjerphan jjerphan Aug 1, 2024

Choose a reason for hiding this comment

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

Is there a preferred way of deprecating import path?

Copy link
Member Author

Choose a reason for hiding this comment

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

New lines come from libmambapy/tests/test_solver_libsolv.py with no modifications.

@jjerphan jjerphan force-pushed the maint/move-libsolv-parameters-up branch from caf5cc1 to 8b2f74a Compare August 1, 2024 10:06
@jjerphan jjerphan marked this pull request as ready for review August 1, 2024 15:01
@jjerphan jjerphan added type::refactoring release::enhancements For enhancements PRs or implementing features labels Aug 27, 2024
@jjerphan jjerphan force-pushed the maint/move-libsolv-parameters-up branch from 8b2f74a to 4cde652 Compare September 12, 2024 05:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release::enhancements For enhancements PRs or implementing features type::refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant