Skip to content

Refactored Dependecies Management#225

Merged
Nusnus merged 4 commits into
celery:mainfrom
Katz-Consulting-Group:bugfix
Mar 4, 2024
Merged

Refactored Dependecies Management#225
Nusnus merged 4 commits into
celery:mainfrom
Katz-Consulting-Group:bugfix

Conversation

@Nusnus
Copy link
Copy Markdown
Member

@Nusnus Nusnus commented Feb 28, 2024

Better fix instead of #213 + CI enhancement to make sure such changes are tested properly

EDIT: Added new extras + doc

@Nusnus Nusnus added the bug Something isn't working label Feb 28, 2024
@Nusnus Nusnus added this to the v1.0.0b3 milestone Feb 28, 2024
@Nusnus Nusnus self-assigned this Feb 28, 2024
@Nusnus Nusnus marked this pull request as ready for review February 28, 2024 17:55
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 28, 2024

Codecov Report

Attention: Patch coverage is 0% with 106 lines in your changes are missing coverage. Please review.

Project coverage is 24.69%. Comparing base (209e658) to head (7bdb294).
Report is 1 commits behind head on main.

Files Patch % Lines
src/pytest_celery/__init__.py 0.00% 91 Missing ⚠️
src/pytest_celery/defaults.py 0.00% 10 Missing ⚠️
src/pytest_celery/vendors/__init__.py 0.00% 5 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #225      +/-   ##
==========================================
- Coverage   26.49%   24.69%   -1.80%     
==========================================
  Files          36       37       +1     
  Lines        1057     1134      +77     
  Branches      213      220       +7     
==========================================
  Hits          280      280              
- Misses        751      828      +77     
  Partials       26       26              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Nusnus Nusnus marked this pull request as draft February 28, 2024 18:40
@Nusnus Nusnus force-pushed the bugfix branch 2 times, most recently from 8ce04ee to c8cf855 Compare March 3, 2024 19:18
@Nusnus Nusnus changed the title Bugfix: Added pytest-celery package to built-in Celery worker Dockerfile TBD Mar 3, 2024
@Nusnus Nusnus force-pushed the bugfix branch 2 times, most recently from a2b511b to 08965f1 Compare March 3, 2024 21:58
@Nusnus Nusnus changed the title TBD Refactored Dependecies Management Mar 4, 2024
@Nusnus Nusnus marked this pull request as ready for review March 4, 2024 17:42
@Nusnus Nusnus merged commit 6575778 into celery:main Mar 4, 2024
Nusnus added a commit to Katz-Consulting-Group/pytest-celery that referenced this pull request Mar 4, 2024
Nusnus added a commit that referenced this pull request Mar 4, 2024
@Nusnus Nusnus deleted the bugfix branch March 5, 2024 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

1 participant