Skip to content

refactor: Move much as possible into new pixi_core crate#4328

Closed
haecker-felix wants to merge 4 commits intoprefix-dev:mainfrom
haecker-felix:pixi_core
Closed

refactor: Move much as possible into new pixi_core crate#4328
haecker-felix wants to merge 4 commits intoprefix-dev:mainfrom
haecker-felix:pixi_core

Conversation

@haecker-felix
Copy link
Collaborator

The idea is to use this as a starting point and to split pixi_core into more crates in following PRs (eg. pixi_environment etc).

As a next step we likely need to introduce some pixi_core_types or similar (inspired by uv_types), as the modules are depending on each other and we very likely would get cyclic dependencie (see #4302 (comment))

Replaces: #4302

@lucascolley lucascolley added refactor Specifies PR or Issue is related to a refactor Rust API labels Aug 13, 2025
@haecker-felix haecker-felix force-pushed the pixi_core branch 3 times, most recently from 942139d to d88a805 Compare August 13, 2025 13:05
Copy link
Contributor

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Only minimal comments, then we can merge it!

@Hofer-Julian Hofer-Julian added the test:extra_slow Run the extra slow tests label Aug 13, 2025
Copy link
Contributor

@Hofer-Julian Hofer-Julian left a comment

Choose a reason for hiding this comment

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

Great work!

@Hofer-Julian
Copy link
Contributor

Close in favor of #4337

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactor Specifies PR or Issue is related to a refactor Rust API test:extra_slow Run the extra slow tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants