Skip to content

Conversation

@iamareebjamal
Copy link
Member

@iamareebjamal iamareebjamal commented May 21, 2019

Part of #5854

@codecov
Copy link

codecov bot commented May 21, 2019

Codecov Report

Merging #5944 into development will increase coverage by 0.12%.
The diff coverage is 83.33%.

Impacted file tree graph

@@               Coverage Diff               @@
##           development    #5944      +/-   ##
===============================================
+ Coverage        66.39%   66.52%   +0.12%     
===============================================
  Files              286      285       -1     
  Lines            13958    13905      -53     
===============================================
- Hits              9268     9250      -18     
+ Misses            4690     4655      -35
Impacted Files Coverage Δ
app/api/helpers/tasks.py 17.59% <100%> (ø) ⬆️
app/__init__.py 86.3% <80%> (+2.21%) ⬆️
app/api/helpers/request_context_task.py

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 35462b7...405966a. Read the comment docs.

@iamareebjamal iamareebjamal changed the title chore: Change requirements syntax chore: Change requirements syntax and upgrade celery May 21, 2019
Copy link
Contributor

@uds5501 uds5501 left a comment

Choose a reason for hiding this comment

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

Only 'Order' import as PDF isn't working (which was broken before too), otherwise other celery tasks are successfully running

@iamareebjamal iamareebjamal merged commit e49d771 into fossasia:development May 22, 2019
@iamareebjamal iamareebjamal deleted the requirements branch May 22, 2019 11:02
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.

2 participants