Skip to content

Cbrash/city optional#118

Merged
shafqatfarhan merged 7 commits into
mainfrom
cbrash/city-optional
Aug 20, 2025
Merged

Cbrash/city optional#118
shafqatfarhan merged 7 commits into
mainfrom
cbrash/city-optional

Conversation

@colinbrash

Copy link
Copy Markdown
Contributor

Merge checklist:
Check off if complete or not applicable:

  • Documentation updated (not only docstrings)
  • Fixup commits are squashed away
  • Unit tests added/updated
  • Manual testing instructions provided
  • Noted any: Concerns, dependencies, migration issues, deadlines, tickets

Post-merge:

@colinbrash colinbrash requested a review from a team as a code owner August 19, 2025 16:31
@github-actions

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  sanctions/apps/api/v1
  views.py
  sanctions/apps/api_client
  sdn_client.py
  sanctions/apps/sanctions
  utils.py 37-48, 88
Project Total  

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

Comment thread sanctions/apps/api/v1/tests/test_views.py Outdated
Comment thread sanctions/apps/api/v1/tests/test_views.py
Comment thread sanctions/apps/api_client/sdn_client.py
@shafqatfarhan shafqatfarhan merged commit 48c6d48 into main Aug 20, 2025
6 checks passed
@shafqatfarhan shafqatfarhan deleted the cbrash/city-optional branch August 20, 2025 15:42
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.

3 participants