Skip to content

fix lint and tests on v1/mvp#308

Merged
braddf merged 4 commits into
v1/mvpfrom
v1-mvp-lint
May 12, 2026
Merged

fix lint and tests on v1/mvp#308
braddf merged 4 commits into
v1/mvpfrom
v1-mvp-lint

Conversation

@peterdudfield
Copy link
Copy Markdown
Contributor

@peterdudfield peterdudfield commented May 12, 2026 •
edited
Loading

Pull Request

Description

Fix linting and tests
helps with #199

How Has This Been Tested?

Ci tests

Checklist:

  • My code follows OCF's coding style guidelines
  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked my code and corrected any misspellings

@peterdudfield peterdudfield changed the title lint fix lint on v1/mvp May 12, 2026
@peterdudfield peterdudfield changed the title fix lint on v1/mvp fix lint and tests on v1/mvp May 12, 2026
@peterdudfield peterdudfield requested a review from braddf May 12, 2026 14:38
braddf
braddf reviewed May 12, 2026
View reviewed changes
Comment thread src/quartz_api/tests/integration/uk_national/test_app.py
@braddf braddf merged commit a68ac28 into v1/mvp May 12, 2026
8 checks passed
@braddf braddf deleted the v1-mvp-lint branch May 12, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@braddf braddf braddf approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@peterdudfield @braddf