Skip to content

Better error message - #837

Merged
mfeurer merged 3 commits into
developfrom
better_errors
Oct 16, 2019
Merged

Better error message#837
mfeurer merged 3 commits into
developfrom
better_errors

Conversation

@mfeurer

Copy link
Copy Markdown
Collaborator

Print the XML if a flow XML does not validate online.

@codecov-io

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (develop@29a023c). Click here to learn what that means.
The diff coverage is 87.5%.

Impacted file tree graph

@@ Coverage Diff @@## develop #837 +/- ##
==========================================
Coverage ? 88.04% ==========================================
Files ? 36 Lines ? 4249 Branches ? 0 ==========================================
Hits ? 3741 Misses ? 508 Partials ? 0
Impacted FilesCoverage Δ
openml/exceptions.py93.54% <100%> (ø)
openml/_api_calls.py83.95% <85.71%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29a023c...d63be16. Read the comment docs.

@amuelleramueller left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@mfeurer
mfeurer merged commit 34d784a into developOct 16, 2019
@mfeurer
mfeurer deleted the better_errors branch October 16, 2019 13:03
Sign up for freeto 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

@mfeurer@codecov-io@amueller