-
Notifications
You must be signed in to change notification settings - Fork 212
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
feat: Create features and environments from extras #1077
Merged
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
9191a03
Create features and environments from extras
olivier-lacroix 4728043
Handle self-refencing extras
olivier-lacroix b6c3f35
Add documentation
olivier-lacroix 46fee3d
lint
olivier-lacroix bbe7d9e
Simplify project templates
olivier-lacroix 551f627
Add a "default" SolveGroup to the default Environment (if not redefined)
olivier-lacroix 6ed29fb
Merge remote-tracking branch 'upstream/main' into pyproject-extras
olivier-lacroix 5fe6b89
Move environment extraction from extras to pyproject.rs
olivier-lacroix c16dea9
Merge branch 'main' into pyproject-extras
olivier-lacroix 1959dfd
Merge branch 'main' into pyproject-extras
olivier-lacroix b7790b9
Merge branch 'main' into pyproject-extras
ruben-arts bd7a8fc
Update docs/advanced/pyproject_toml.md
olivier-lacroix 9077bdd
Revert "Add a "default" SolveGroup to the default Environment (if not…
olivier-lacroix 8ca31db
Merge remote-tracking branch 'upstream/main' into pyproject-extras
olivier-lacroix 3f116d9
refactor parsing and display info message
olivier-lacroix b4c6e9d
revert test changes
olivier-lacroix 1c2f5f1
lint
olivier-lacroix 54d6772
Refine message, and update flask example
olivier-lacroix d09885a
Simplify print
olivier-lacroix 05650ee
Add a message when not runnning init
olivier-lacroix d6e9833
cleanup
olivier-lacroix 0ee0411
Filter out unused features
olivier-lacroix File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
# GitHub syntax highlighting | ||
pixi.lock linguist-language=YAML |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1 +1,3 @@ | ||
# pixi environments | ||
.pixi | ||
*.egg-info |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
With the above, is it correct that installing the
test
environment withpixi
would installpytest
frompypi
rather thanconda-forge
?If I wanted my
pixi
test
environment to only install dependencies fromconda-forge
can I specify that?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@dhirschfeld you are correct.
Currently, you would have to manually add them to the
tool.pixi.feature.test.dependencies
table. What you want is also on my wishlist :-) #532There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So, if I did manually specify the test dependencies in
[tool.pixi.feature.test.dependencies]
table, then that would be used instead of mappingpypi
dependencies from the[project.optional-dependencies]
table?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. Pixi currently solves pypi dependencies after the conda ones. And if it detects a package is already provided by one of the conda packages requested, it is skipped.