Skip to content

Hotfix: add pytz to api "prod" requirements#2819

Open
NatSquared wants to merge 2 commits intomainfrom
hotfix-api-prod-requirements-add-pytz
Open

Hotfix: add pytz to api "prod" requirements#2819
NatSquared wants to merge 2 commits intomainfrom
hotfix-api-prod-requirements-add-pytz

Conversation

@NatSquared
Copy link
Copy Markdown
Collaborator

@NatSquared NatSquared commented Apr 9, 2026

Issue #: N/A

Description of changes:
Hotfix an issue in cron where pytz was not being installed, causing crashes like the below.
This is because in api/setup.py, only prod.txt is installed as requirements for the api package.
Also upgrade pytz to 2026.1.post1 globally for consistency (BC has a new time zone!)

image

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.94%. Comparing base (a4e4b1c) to head (257e5c2).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2819      +/-   ##
==========================================
+ Coverage   75.43%   75.94%   +0.50%     
==========================================
  Files         519      556      +37     
  Lines       19877    20645     +768     
  Branches     1568     1568              
==========================================
+ Hits        14995    15678     +683     
- Misses       4876     4961      +85     
  Partials        6        6              
Flag Coverage Δ
analyticsapi 88.93% <ø> (?)
api 86.05% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
see 37 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sonarqubecloud
Copy link
Copy Markdown

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.

2 participants