Conversation
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
Signed-off-by: Thomas Poignant <thomas.poignant@gofeatureflag.org>
| - name: Checkout repository | ||
| uses: actions/checkout@v4 | ||
| - name: OpenAPI Lint Checks | ||
| uses: nwestfall/openapi-action@v1.0.2 |
There was a problem hiding this comment.
Regarding this action, it seems this has been unmaintained for some time. And I checked alternatives and came across redocly-cli (MIT license) and used their docker image [1] to perform a similar validation
You can see my attempt in the fork [2] and the test GH action failure run in my fork [3]
Let me know your thoughts on this :)
[1] - https://github.com/Redocly/redocly-cli?tab=readme-ov-file#docker
[2] - https://github.com/open-feature/protocol/compare/test-validate
[3] - https://github.com/open-feature/protocol/actions/runs/8913421964/job/24478855581
There was a problem hiding this comment.
This is another option that adds lint violations as annotations in the PR.
There was a problem hiding this comment.
@beeme1mr checked but I am not super happy yet as there are open issues and PRs that were not addressed recently. We can switch back if we see issues with redocly
|
@Kavindu-Dodan Maybe we should take your branch instead of this one. |
I will take a shot on this then :) |
|
@thomaspoignant here is a PR with redocly - #15 |
|
Close this PR in favor of #15 |
Add linter for OpenAPI validation.