-
Notifications
You must be signed in to change notification settings - Fork 16
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
allow passing a path to activate #38
Comments
I've never fully worked out how exactly it |
Here is another motivation to support paths. The recommended way to share a project between Pluto notebooks is Pkg.activate(Base.current_project()) So it seems natural to replace that call with begin
using TestEnv
base_project = Base.current_project()
Pkg.activate(base_project); # until Pluto knows that TestEnv.activate should disable Pluto pkg system
TestEnv.activate(base_project)
end which currently fails with TestEnv.TestEnvError("/path/to/package/Project.toml not installed 👻")
ctx_and_pkgspec(::String)@common.jl:24
activate(::String)@activate_set.jl:10
top-level scope@[Local: 5](http://localhost:1234/edit?id=fffa3cba-43f3-11ee-1d2b-db91fbb12088#) removing the begin
using TestEnv
base_project = Base.current_project()
Pkg.activate(base_project); # until Pluto knows that TestEnv.activate should disable Pluto pkg system
TestEnv.activate()
end The issue was that TestEnv.jl/src/julia-1.9/common.jl Line 22 in 0b9a63e
In other words, |
Food for thoughts after a shallow analysis (feeling out of my league here), After a look into
pro: uses the regular |
I think it would be much more convenient if one can pass a path instead of an available package name to create the test environment since
path
is a more straightforward concept that everyone understands, e.g in a hierarchical project like https://github.com/QuantumBFS/Yao.jl/ we have a lot of component packages live inlib
directory, and packages inlib
do not necessarily to be listed as a dependency in theProject.toml
of upper-level packages, which as a result usingname
will sayxxx
isn't installed. But I think it's reasonable to activate a random package locally from a shared environment that hasTestEnv
installed.The text was updated successfully, but these errors were encountered: