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

better info messages for activating new env #618

Merged
merged 1 commit into from
Aug 13, 2018
Merged

better info messages for activating new env #618

merged 1 commit into from
Aug 13, 2018

Conversation

fredrikekre
Copy link
Member

No description provided.

@Evizero
Copy link
Contributor

Evizero commented Aug 11, 2018

much better

src/API.jl Outdated
@@ -611,7 +611,7 @@ function activate(path::String; shared::Bool=false)
# unless the shared environment already exists, place it in the first depots
if !isdir(fullpath)
fullpath = joinpath(Pkg.envdir(Pkg.depots1()), path)
@info("new shared environment \"$path\" will be placed at $fullpath")
@info("activating new shared environment \"$path\" at $fullpath.")
Copy link
Member

Choose a reason for hiding this comment

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

Even though we use --shared I don't think we should call it shared environment. It's just a normal environment, no?

@KristofferC
Copy link
Member

Looks like CI need to be fixed.

@Evizero
Copy link
Contributor

Evizero commented Aug 13, 2018

Oh yea, I added tests for the log messages, which will need to be updated

@fredrikekre
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Aug 13, 2018
618: better info messages for activating new env r=fredrikekre a=fredrikekre



631: stop testing on 0.7 r=KristofferC a=KristofferC

No point

634: fix gc in prescence of normal files r=KristofferC a=KristofferC

Fixes #601 

Co-authored-by: Fredrik Ekre <fredrik.ekre@chalmers.se>
Co-authored-by: Kristoffer Carlsson <kcarlsson89@gmail.com>
@bors
Copy link
Contributor

bors bot commented Aug 13, 2018

Canceled (will resume)

bors bot added a commit that referenced this pull request Aug 13, 2018
618: better info messages for activating new env r=fredrikekre a=fredrikekre



Co-authored-by: Fredrik Ekre <fredrik.ekre@chalmers.se>
@KristofferC
Copy link
Member

bors r-

@bors
Copy link
Contributor

bors bot commented Aug 13, 2018

Canceled

@KristofferC
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Aug 13, 2018
599: Remove `--env` meta option r=KristofferC a=00vareladavid

Should close #590 and close #549

618: better info messages for activating new env r=KristofferC a=fredrikekre



Co-authored-by: David Varela <00.varela.david@gmail.com>
Co-authored-by: Fredrik Ekre <fredrik.ekre@chalmers.se>
@bors
Copy link
Contributor

bors bot commented Aug 13, 2018

Timed out (retrying...)

@KristofferC
Copy link
Member

bors r-

@bors
Copy link
Contributor

bors bot commented Aug 13, 2018

Canceled

@KristofferC
Copy link
Member

bors r+

bors bot added a commit that referenced this pull request Aug 13, 2018
599: Remove `--env` meta option r=KristofferC a=00vareladavid

Should close #590 and close #549

618: better info messages for activating new env r=KristofferC a=fredrikekre



Co-authored-by: David Varela <00.varela.david@gmail.com>
Co-authored-by: Fredrik Ekre <fredrik.ekre@chalmers.se>
@bors
Copy link
Contributor

bors bot commented Aug 13, 2018

@bors bors bot merged commit be4d03d into master Aug 13, 2018
@fredrikekre fredrikekre deleted the fe/msg branch August 13, 2018 17:43
@KristofferC KristofferC mentioned this pull request Aug 20, 2018
22 tasks
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