Pass CI tests on main

[Dhananjhay]

Proposed changes

Describe the changes implemented in this pull request. If the changes fix a bug or resolves a feature request, be sure to link the issue (and explain how the changes address the issue). Be sure to select a label describing the PR (e.g. maintenance).

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you are unsure about any of the choices, don't hesitate to ask!

• Changes have been tested to ensure that fix is effective or that a feature works.
• Changes pass the unit tests
• I have included necessary documentation or comments (as necessary)
• Any dependent changes have been merged and published

Notes

All PRs will undergo the unit testing before being reviewed. You may be requested to explain or make additional changes before the PR is accepted.

[akhanf]

Thanks!

[Dhananjhay]

All 37 unit tests passed! Ready to merge @akhanf.

[akhanf]

Is that commit message correct? I didn't think bids specs are being updated here..

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.37%. Comparing base (ffb7752) to head (ff5bbc9).
Report is 3 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #465   +/-   ##
=======================================
  Coverage   94.37%   94.37%           
=======================================
  Files          55       55           
  Lines        2222     2223    +1     
=======================================
+ Hits         2097     2098    +1     
  Misses        125      125           

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

[Dhananjhay]

Yes. I was hesitant with the commit message too but the error on main branch said bids specs out of date, run poetry run poe update-bids but I've fixed the message now; it was merely a syntax issue.