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

Make ExceptionInfo.repr change more prominent #5648

Merged
merged 1 commit into from Jul 24, 2019

Conversation

nicoddemus
Copy link
Member

Related to #5579

@codecov
Copy link

codecov bot commented Jul 23, 2019

Codecov Report

Merging #5648 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5648   +/-   ##
=======================================
  Coverage   96.12%   96.12%           
=======================================
  Files         117      117           
  Lines       25770    25770           
  Branches     2495     2495           
=======================================
  Hits        24771    24771           
  Misses        695      695           
  Partials      304      304

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 13d8577...3c94f32. Read the comment docs.

@stadelmanma
Copy link

Looks good! That is all I would have needed to see to realize the problem.

@nicoddemus nicoddemus merged commit 7fa2748 into pytest-dev:master Jul 24, 2019
@nicoddemus nicoddemus deleted the improve-exc-info-changelog branch July 24, 2019 00:24
@nicoddemus
Copy link
Member Author

Looks good! That is all I would have needed to see to realize the problem.

Sorry, that's on us for 1) don't realize it would be such a problem, 2) don't realize it wouldn't be abvious.

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.

None yet

3 participants