Skip to content

docs: Provide guidance on using direnv#3513

Merged
Hofer-Julian merged 17 commits intoprefix-dev:mainfrom
phreed:main
Apr 11, 2025
Merged

docs: Provide guidance on using direnv#3513
Hofer-Julian merged 17 commits intoprefix-dev:mainfrom
phreed:main

Conversation

@phreed
Copy link
Copy Markdown
Contributor

@phreed phreed commented Apr 4, 2025

Specifically, guidance on using direnv in the context of IDE's

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The Python extension also has proper pixi support, we should mention that IMO

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

You mean this https://code.visualstudio.com/docs/python/python-quick-start extension, correct?
I do not see an explicit mention of pixi.
I suppose it is something similar to conda?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

yes it's this extension. maybe add an image guide on how to do it?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yeah, If we have a proper vscode section, the Python extension should be mentioned as well 🙂

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am not entirely sure how to use the python extension with pixi.
I saw that there is some effort to make a pixi conda but I do not think that is the way to go.
I am looking for a way to use conda run ... as that should be translatable to pixi run ....
But, I am not finding that.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I suppose trampolines could be used as the target of
image
but, trampolines are not provided as part of project workspaces.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I suppose it comes down to the settings.yml for the python plugin:

https://code.visualstudio.com/docs/python/settings-reference

It is not obvious to me how this should be configured.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It should no configuration.

I just created a fresh Pixi workspace with Python, started vscode in it and vscode immediately selected the pixi environment. Maybe @baszalmstra can check if something is broken on Windows at the moment?

@phreed phreed changed the title Provide guidance on using direnv docs: Provide guidance on using direnv Apr 7, 2025
Copy link
Copy Markdown
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.

Looks good at first glance, will try to find time to give a proper review later

Copy link
Copy Markdown
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.

I've cleaned things up a bit, and I think it looks good now.

Thanks for your contribution @phreed!

@Hofer-Julian Hofer-Julian enabled auto-merge (squash) April 11, 2025 11:48
@Hofer-Julian Hofer-Julian merged commit 56a433f into prefix-dev:main Apr 11, 2025
35 checks passed
@phreed
Copy link
Copy Markdown
Contributor Author

phreed commented Apr 11, 2025

The combination of the VSCode stuff into a single page makes sense, as VSCode is a single application.
I have found that the isolation provided by pixi is sufficient to make use of devcontainer (on linux at least) unnecessary.

I am not so sure about combining the Jetbrains stuff into a single page though, PyCharm, Idea.
They are separate applications produced by the same vendor.
I should have called the new page for Jetbrains/IDEA, idea.md rather than jetbrains.md.

@Hofer-Julian
Copy link
Copy Markdown
Contributor

I am not so sure about combining the Jetbrains stuff into a single page though, PyCharm, Idea.
They are separate applications produced by the same vendor.
I should have called the new page for Jetbrains/IDEA, idea.md rather than jetbrains.md.

I see your point, for now I think it's fine. If more content comes up where the JetBrains products differ, we can split them again

@phreed
Copy link
Copy Markdown
Contributor Author

phreed commented Apr 11, 2025

I did start a branch to capture the point about jetbrains products.
main...phreed:pixi:jetbrains-docs
That said I guess this MR is done.

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.

3 participants