Skip to content

Merge pull request #3227 from hmaarrfk/travis_cleanup_pip (solve np.lookfor for v0.14.x)#3555

Closed
sciunto wants to merge 0 commit intoscikit-image:v0.14.xfrom
sciunto:v0.14.x
Closed

Merge pull request #3227 from hmaarrfk/travis_cleanup_pip (solve np.lookfor for v0.14.x)#3555
sciunto wants to merge 0 commit intoscikit-image:v0.14.xfrom
sciunto:v0.14.x

Conversation

@sciunto
Copy link
Copy Markdown
Member

@sciunto sciunto commented Nov 17, 2018

Travis-CI: cleanup pip commands

Description

See #3543

@sciunto sciunto requested a review from jni November 17, 2018 19:54
@sciunto
Copy link
Copy Markdown
Member Author

sciunto commented Nov 17, 2018

ping @hmaarrfk Can you check this please? I had a hard time this git...

@sciunto sciunto changed the title Merge pull request #3227 from hmaarrfk/travis_cleanup_pip Merge pull request #3227 from hmaarrfk/travis_cleanup_pip (solve np.lookfor for v0.14.x) Nov 17, 2018
@sciunto sciunto added this to the 0.14.2 milestone Nov 17, 2018
@sciunto
Copy link
Copy Markdown
Member Author

sciunto commented Nov 17, 2018

The cython version looks correct now. But we are facing the same issues as in master for which I have no idea of the origin... We are in a warfare.

@sciunto
Copy link
Copy Markdown
Member Author

sciunto commented Nov 19, 2018

warefare must be fixed by #3557

@hmaarrfk
Copy link
Copy Markdown
Member

I think maybe you can rebase this now?

@sciunto
Copy link
Copy Markdown
Member Author

sciunto commented Nov 19, 2018

oooppsss, i didn't want to push directly to the repo... :( hopefully, it is ok. My bad...

@jni
Copy link
Copy Markdown
Member

jni commented Nov 19, 2018

Erm, what just happened...

@jni
Copy link
Copy Markdown
Member

jni commented Nov 19, 2018

Just looked at the commits, looks like you merged without review, which isn't a huge deal. =)

@sciunto sciunto deleted the v0.14.x branch November 20, 2018 12:46
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