Closed patphongs closed 4 months ago
Attention: Patch coverage is 0%
with 18 lines
in your changes are missing coverage. Please review.
Project coverage is 77.84%. Comparing base (
cf19bed
) to head (af07dec
). Report is 2 commits behind head on develop.:exclamation: Current head af07dec differs from pull request most recent head 3999cbd
Please upload reports for the commit 3999cbd to get more accurate results.
Files | Patch % | Lines |
---|---|---|
fec/legal/views.py | 0.00% | 18 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Closing due to failed package-lock.json
Summary (required)
(Include a summary of proposed changes and connect issue below)
Required reviewers
(Include who is required to review prior to merge. For example: One designer and two front end developer reviews are required prior to merge)
Impacted areas of the application
General components of the application that this PR will affect:
Screenshots
(Include a screenshot of the new/updated features in context (“in the wild”). If it is an interface change, include both before and after screenshots)
Related PRs
Related PRs against other branches:
How to test
(Include any information that may be helpful to the reviewer(s). This might include links to sample pages to test or any local environmental setup that is unusual such as environment variable (never credentials), API version to point to, etc)
System architecture updates (if applicable)
(If this pull request changes our current system diagram, include a description of those changes here and create a new ticket to update the system diagram)