Skip to content

fix openapi 3.0 security schemes import#7822

Merged
calvernaz merged 1 commit intomasterfrom
fix-security-schemes-openapi3
Mar 11, 2023
Merged

fix openapi 3.0 security schemes import#7822
calvernaz merged 1 commit intomasterfrom
fix-security-schemes-openapi3

Conversation

@calvernaz
Copy link
Contributor

@calvernaz calvernaz commented Mar 8, 2023

@calvernaz calvernaz temporarily deployed to localstack-ext-tests March 8, 2023 20:58 — with GitHub Actions Inactive
@calvernaz calvernaz requested review from joe4dev and whummer March 8, 2023 21:33
@github-actions
Copy link

github-actions bot commented Mar 8, 2023

LocalStack integration with Pro

       3 files  ±0         3 suites  ±0   1h 31m 55s ⏱️ -45s
1 788 tests ±0  1 405 ✔️  - 1  383 💤 +1  0 ±0 
2 514 runs  ±0  1 779 ✔️  - 1  735 💤 +1  0 ±0 

Results for commit 1f63223. ± Comparison against base commit e9c2ce5.

♻️ This comment has been updated with latest results.

@calvernaz calvernaz temporarily deployed to localstack-ext-tests March 9, 2023 09:19 — with GitHub Actions Inactive
Copy link
Member

@whummer whummer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍 (once the tests are passing..) Re-triggered the CI pipeline, but you may have to rebase onto latest master

@calvernaz calvernaz force-pushed the fix-security-schemes-openapi3 branch from 4a15278 to 1f63223 Compare March 10, 2023 21:27
@calvernaz calvernaz temporarily deployed to localstack-ext-tests March 10, 2023 21:27 — with GitHub Actions Inactive
@coveralls
Copy link

Coverage Status

Coverage: 85.035% (-0.04%) from 85.07% when pulling 1f63223 on fix-security-schemes-openapi3 into e9c2ce5 on master.

@calvernaz calvernaz temporarily deployed to localstack-ext-tests March 10, 2023 23:19 — with GitHub Actions Inactive
@calvernaz calvernaz merged commit a5a5ba8 into master Mar 11, 2023
@calvernaz calvernaz deleted the fix-security-schemes-openapi3 branch March 11, 2023 08:05
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