Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add github action to generate single API spec #43

Merged
merged 12 commits into from
Feb 27, 2024

Conversation

dvacca-onfido
Copy link
Contributor

@dvacca-onfido dvacca-onfido commented Feb 26, 2024

Initial version of Github Action workflow to build and push single-file OpenAPI specification.

Include validation of multi file specification and a few files which were forgotten in PR#42.

@dvacca-onfido dvacca-onfido self-assigned this Feb 26, 2024
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch 3 times, most recently from 3d14db7 to 8085b6c Compare February 26, 2024 14:30
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch 7 times, most recently from f180d74 to f1b998e Compare February 26, 2024 14:57
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from f1b998e to 0a25c96 Compare February 26, 2024 14:59
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch 9 times, most recently from 94cfa62 to 32675d0 Compare February 26, 2024 16:08
openapi.yaml Outdated
@@ -5,6 +5,7 @@ info:
version: 3.6.0
contact:
name: Onfido
url: https://onfido.com
email: [email protected]
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be support's email actually

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated

@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from 32675d0 to 5c86ad1 Compare February 26, 2024 17:07
@@ -1,6 +1,6 @@
# Onfido OpenAPI specification (beta)

:warning: Our OpenAPI specification is currently in beta and we welcome any feedback. You can contact us via the issues tab or [email](mailto:openapi-feedback@onfido.com), but we don't yet officially support this specification. :warning:
:warning: Our OpenAPI specification is currently in beta and we welcome any feedback. You can contact us via the issues tab or [support portal](https://public.support.onfido.com), but we don't yet officially support this specification. :warning:
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we consider that the spec is out of beta since we have all the reports ?

Copy link
Contributor Author

@dvacca-onfido dvacca-onfido Feb 26, 2024

Choose a reason for hiding this comment

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

I believe we should wait a little bit longer, to have at least one (generated) client library sufficiently tested to be definition maps quite well the actual API.

@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from 4a03471 to 70fdb3d Compare February 26, 2024 17:20
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from 70fdb3d to cab6e9c Compare February 26, 2024 17:25
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from cab6e9c to e83b112 Compare February 26, 2024 17:27
@dvacca-onfido dvacca-onfido marked this pull request as ready for review February 26, 2024 17:29
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch 2 times, most recently from c14e8e3 to 7bc9de5 Compare February 26, 2024 17:56
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch 3 times, most recently from 364654a to aae5c88 Compare February 27, 2024 08:54
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from aae5c88 to ecbf6fd Compare February 27, 2024 08:56
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch 3 times, most recently from 9ded090 to 38ce035 Compare February 27, 2024 09:26
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from 38ce035 to d6fd159 Compare February 27, 2024 09:27
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch 2 times, most recently from ee5a452 to 009c499 Compare February 27, 2024 09:32
@dvacca-onfido dvacca-onfido force-pushed the generate-single-file-specs branch from 009c499 to c63f0ae Compare February 27, 2024 09:33
@dvacca-onfido dvacca-onfido merged commit 09b2a72 into master Feb 27, 2024
2 checks passed
@dvacca-onfido dvacca-onfido deleted the generate-single-file-specs branch March 8, 2024 08:07
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