Skip to content

Cbrash/city optional#118

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

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

Conversation

@colinbrash
Copy link
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

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

@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