Skip to content

Add _warnings #1019

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

Merged
merged 2 commits into from
Jun 7, 2019
Merged

Add _warnings #1019

merged 2 commits into from
Jun 7, 2019

Conversation

palaviv
Copy link
Contributor

@palaviv palaviv commented Jun 7, 2019

This is just a the _warnings module and a warn method without implementation so I can continue with #958.

Co-Authored-By: coolreader18 <33094578+coolreader18@users.noreply.github.com>
@codecov-io
Copy link

Codecov Report

Merging #1019 into master will increase coverage by 0.01%.
The diff coverage is 88.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1019      +/-   ##
==========================================
+ Coverage   64.57%   64.59%   +0.01%     
==========================================
  Files          96       97       +1     
  Lines       16932    16950      +18     
  Branches     3780     3781       +1     
==========================================
+ Hits        10934    10949      +15     
  Misses       3438     3438              
- Partials     2560     2563       +3
Impacted Files Coverage Δ
vm/src/stdlib/mod.rs 100% <100%> (ø) ⬆️
vm/src/stdlib/warnings.rs 87.5% <87.5%> (ø)
vm/src/function.rs 56.69% <0%> (-0.26%) ⬇️

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 1b7088c...8fccad1. Read the comment docs.

@coolreader18 coolreader18 merged commit d27840c into RustPython:master Jun 7, 2019
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