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.

Sign up for GitHub

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

Jump to bottom

Add explicit error message if users use the django app before .ready() #955

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Add explicit error message if users use the django app before .ready() #955

merged 1 commit into from
Feb 29, 2024

Conversation

ewjoachim
Copy link
Member

@ewjoachim ewjoachim commented Feb 29, 2024 •
edited
Loading

Relates to #934

Successful PR Checklist:

  • Tests
    • (not applicable?)
  • Documentation
    • (not applicable?)

PR label(s):

@ewjoachim ewjoachim requested a review from a team as a code owner February 29, 2024 22:36
@github-actions github-actions bot added the PR type: bugfix 🕵️ Contains bug fix label Feb 29, 2024
@github-actions GitHub Actions
Copy link

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)		Lines missing
  procrastinate
  app.py
  blueprints.py
  tasks.py
  procrastinate/contrib/django
  __init__.py
  exceptions.py
  placeholder.py
Project Total  

This report was generated by python-coverage-comment-action

@ewjoachim ewjoachim merged commit 445ad89 into main Feb 29, 2024
13 checks passed
@ewjoachim ewjoachim deleted the pre-app-django branch February 29, 2024 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
PR type: bugfix 🕵️ Contains bug fix
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@ewjoachim