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

document how to use at_test_logs to check for no warnings #39025

Merged
merged 3 commits into from
Apr 16, 2021

Conversation

ranocha
Copy link
Member

@ranocha ranocha commented Dec 29, 2020

This documents how to use @test_logs to check for no warnings. Additionally, I added Logging. to the min_levels to make it more clear (since min_level=Warn does not work on Julia v1.5, even after using Logging).

@ranocha
Copy link
Member Author

ranocha commented Jan 5, 2021

I don't see how the changes of this PR could cause the test failure.

@ranocha ranocha closed this Feb 10, 2021
@ranocha ranocha reopened this Feb 10, 2021
@ranocha
Copy link
Member Author

ranocha commented Feb 10, 2021

An appropriate label would be doc (but I can't add labels).

@ranocha
Copy link
Member Author

ranocha commented Feb 23, 2021

Bump. Is there any interest in this?

@vtjnash
Copy link
Member

vtjnash commented Apr 16, 2021

It looks like something similar was added in #39670, but I thought the extra details here were useful. I've taken the liberty of combining them and refining the text some more for tone.

@vtjnash vtjnash added docs This change adds or pertains to documentation merge me PR is reviewed. Merge when all tests are passing labels Apr 16, 2021
stdlib/Test/src/logging.jl Outdated Show resolved Hide resolved
@vtjnash vtjnash merged commit d41fac3 into JuliaLang:master Apr 16, 2021
@simeonschaub simeonschaub removed the merge me PR is reviewed. Merge when all tests are passing label Apr 28, 2021
ElOceanografo pushed a commit to ElOceanografo/julia that referenced this pull request May 4, 2021
antoine-levitt pushed a commit to antoine-levitt/julia that referenced this pull request May 9, 2021
johanmon pushed a commit to johanmon/julia that referenced this pull request Jul 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants