Skip to content

Conversation

KristofferC
Copy link
Member

apr-04-2018 15-08-29

  • Show in the Pkg3 prompt the name of the current project (also shown in gif above). Some users have been a bit confused about what their current project is, this should make that more explicit.

@ararslan
Copy link
Member

ararslan commented Apr 4, 2018

Weird, I can't reproduce that FreeBSD failure locally. @iblis17, have you seen that before?

@iblislin
Copy link
Member

iblislin commented Apr 5, 2018

I saw it on recent master build. It only happened on one of worker. I'm going to detach it and check.

@iblislin
Copy link
Member

iblislin commented Apr 5, 2018

@ararslan found a way to reproduce

$ touch /tmp/test.magic
$ gmake -C test REPL

That test case is somewhat ... fragile.
Will sent a PR later.

@StefanKarpinski
Copy link
Member

@iblis17: is the test case relevant to this PR or can I go ahead and merge this?

@StefanKarpinski
Copy link
Member

Also, @KristofferC, is this good to go from your perspective?

@KristofferC
Copy link
Member Author

Looks like there are some cfunction deprecations so hold off with merging a bit.

@iblislin
Copy link
Member

iblislin commented Apr 5, 2018

Is the test case relevant to this PR or can I go ahead and merge this?

That test case is irrelevant, and it has been solved. No idea why only BSD CI failed now.

@KristofferC
Copy link
Member Author

Because of new merges to master.

KristofferC and others added 4 commits April 5, 2018 21:41
* show a progress bar when cloning git repos and add credential handling

(cherry picked from commit 1f8b80aa0942c9d481da7bb87646d02f6546d7f7)
(cherry picked from commit 5755370521a00d2c72d5388aa64b86605d6506d5)
(cherry picked from commit 88811b4091e3606fdd4a86d67538f32693345ee7)
@KristofferC KristofferC merged commit de26816 into master Apr 6, 2018
@KristofferC KristofferC deleted the kc/bump_pkg3_4 branch April 6, 2018 06:54
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.

4 participants