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

chore: Remove flask limiter warning #6658

Merged
merged 1 commit into from Dec 13, 2019

Conversation

iamareebjamal
Copy link
Member

@iamareebjamal iamareebjamal commented Dec 13, 2019

Remove warning for default usage of get_ipaddr function

@auto-label auto-label bot added the chore label Dec 13, 2019
@codecov
Copy link

codecov bot commented Dec 13, 2019

Codecov Report

Merging #6658 into development will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #6658      +/-   ##
===============================================
+ Coverage        65.33%   65.33%   +<.01%     
===============================================
  Files              298      298              
  Lines            15243    15242       -1     
===============================================
  Hits              9959     9959              
+ Misses            5284     5283       -1
Impacted Files Coverage Δ
app/api/custom/orders.py 41.26% <ø> (-0.92%) ⬇️
app/api/auth.py 23.82% <ø> (-0.33%) ⬇️
app/__init__.py 88.05% <100%> (+0.05%) ⬆️
app/models/event.py 78.92% <0%> (+0.35%) ⬆️

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 80b47a6...c246d56. Read the comment docs.

@iamareebjamal iamareebjamal merged commit 9a2375a into fossasia:development Dec 13, 2019
@iamareebjamal iamareebjamal deleted the flask-limiter branch December 13, 2019 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant