Skip to content

Novice skip url test#3320

Merged
jni merged 2 commits intoscikit-image:masterfrom
hmaarrfk:novice_disable_https_test
Aug 5, 2018
Merged

Novice skip url test#3320
jni merged 2 commits intoscikit-image:masterfrom
hmaarrfk:novice_disable_https_test

Conversation

@hmaarrfk
Copy link
Copy Markdown
Member

@hmaarrfk hmaarrfk commented Aug 5, 2018

It was failing anyway on OSX. Novice is deprecated and is causing the rest of the build to halt early.

I tried to XFAIL it, but apparently that isn't implemented for pytest dectests.

Checklist

[It's fine to submit PRs which are a work in progress! But before they are merged, all PRs should provide:]

[For detailed information on these and other aspects see scikit-image contribution guidelines]

References

[If this is a bug-fix or enhancement, it closes issue # ]
[If this is a new feature, it implements the following paper: ]

For reviewers

(Don't remove the checklist below.)

  • Check that the PR title is short, concise, and will make sense 1 year
    later.
  • Check that new functions are imported in corresponding __init__.py.
  • Check that new features, API changes, and deprecations are mentioned in
    doc/release/release_dev.rst.
  • Consider backporting the PR with @meeseeksdev backport to v0.14.x

@pep8speaks
Copy link
Copy Markdown

Hello @hmaarrfk! Thanks for submitting the PR.

Line 217:80: E501 line too long (96 > 79 characters)

Copy link
Copy Markdown
Member

@jni jni left a comment

Choose a reason for hiding this comment

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

Green! 🎉

@jni jni merged commit d3b8b6c into scikit-image:master Aug 5, 2018
@hmaarrfk
Copy link
Copy Markdown
Member Author

@meeseeksdev backport to v0.14.x

@lumberbot-app
Copy link
Copy Markdown

lumberbot-app bot commented Aug 15, 2018

Awww, sorry hmaarrfk you do not seem to be allowed to do that, please ask a repository maintainer.

@jni
Copy link
Copy Markdown
Member

jni commented Aug 15, 2018

@meeseeksdev backport to v0.14.x

lumberbot-app bot pushed a commit that referenced this pull request Aug 15, 2018
hmaarrfk added a commit to hmaarrfk/scikit-image that referenced this pull request Aug 15, 2018
@hmaarrfk hmaarrfk deleted the novice_disable_https_test branch November 4, 2018 13:30
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