Skip to content

Fix charge density task_ids list query#707

Merged
munrojm merged 2 commits intomainfrom
bugfix/charge_density_task_ids
Nov 14, 2022
Merged

Fix charge density task_ids list query#707
munrojm merged 2 commits intomainfrom
bugfix/charge_density_task_ids

Conversation

@munrojm
Copy link
Copy Markdown
Member

@munrojm munrojm commented Nov 14, 2022

Fixes a bug where task_id list queries were not getting formatted properly for the charge density related methods.

@munrojm munrojm added the release:patch Patch release label Nov 14, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Nov 14, 2022

Codecov Report

❌ Patch coverage is 57.14286% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.23%. Comparing base (17dd2cc) to head (5cb5e66).
⚠️ Report is 540 commits behind head on main.

Files with missing lines Patch % Lines
mp_api/client/routes/charge_density.py 57.14% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #707      +/-   ##
==========================================
+ Coverage   81.18%   81.23%   +0.04%     
==========================================
  Files          36       36              
  Lines        1733     1737       +4     
==========================================
+ Hits         1407     1411       +4     
  Misses        326      326              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@munrojm munrojm merged commit 574cda5 into main Nov 14, 2022
@munrojm munrojm deleted the bugfix/charge_density_task_ids branch November 14, 2022 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:patch Patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants