Skip to content

Fix OpenAPI client when Path variables contains upper case - #265

Merged
davorrunje merged 22 commits into
mainfrom
api-issue
Sep 24, 2024
Merged

davorrunje merged 22 commits into
mainfrom
api-issue

Conversation

@rjambrecic

Copy link
Copy Markdown
Contributor

Description

Please include a summary of the change and specify which issue is being addressed. Additionally, provide relevant motivation and context.

Fixes # (issue number)

Type of change

Please delete options that are not relevant.

  • Bug fix (a non-breaking change that resolves an issue)

Checklist

  • My code adheres to the style guidelines of this project (scripts/lint.sh shows no errors)
  • I have conducted a self-review of my own code
  • I have made the necessary changes to the documentation
  • My changes do not generate any new warnings
  • I have added tests to validate the effectiveness of my fix or the functionality of my new feature
  • Both new and existing unit tests pass successfully on my local environment by running scripts/test-cov.sh
  • I have ensured that static analysis tests are passing by running scripts/static-anaylysis.sh
  • I have included code examples to illustrate the modifications

Comment thread fastagency/api/openapi/client.py
@codecov

codecov Bot commented Sep 24, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 80.76923% with 5 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
fastagency/api/openapi/client.py 80.76% 2 Missing and 3 partials ⚠️
Files with missing lines Coverage Δ
fastagency/api/openapi/client.py 87.14% <80.76%> (-1.46%) ⬇️

@davorrunje
davorrunje added this pull request to the merge queue Sep 24, 2024
Merged via the queue into main with commit 54d67e3 Sep 24, 2024
@davorrunje
davorrunje deleted the api-issue branch September 24, 2024 07:08
@rjambrecic rjambrecic linked an issue Sep 24, 2024 that may be closed by this pull request
2 tasks

This branch was previously deployed

1 inactive deployment
testing — c90fda26 Deployed Sep 24, 2024 by davorrunje
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.

Fix OpenAPI client for endpoints with Path variables

3 participants