-
Notifications
You must be signed in to change notification settings - Fork 117
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
Need automated Open API syntax validation on submitting PR to master #366
Comments
@ravi-prakash-v This seems a fairly easy task, assuming that we can use this library & the GitHub actions can be configured easily as well. I have tried to find the guide for local setup but couldn't find one, can you please point me to it? And also feel free to assign it to me, if no one else is working on it. |
we can also use https://github.com/IBM/openapi-validator . as a cli tool to validate open api syntaxes |
@Sarfraz-droid I will take a look at it. |
@ravi-prakash-v Just checked the GHA marketplace, there are some readily available OAS validator GitHub actions like this. Will this do? ( If yes, I will raise a PR ) or would you like to build the entire thing from scratch? |
Hi!
Without these details, the ticket cannot be listed on the C4GT Community Listing. Please update the ticket |
Hi @Deekshithrathod, that will work. Can you please raise a PR for this? |
@ravi-prakash-v Sure can do. Feel free to assign this to me. |
@ravi-prakash-v Raised a PR, do check & let me know if you have any suggestions. |
Hi @ravi-prakash-v , the PR has been raised for the above issue by @Deekshithrathod . Please find it here - #387 |
@ravi-prakash-v , Can I work on this issue , I believe the skills of this project align to my skills. I would like to work on this issue. |
@ravi-prakash-v I understand the problem, and I believe I have the necessary skills. Could I have the opportunity to work on this? |
Hello @ravi-prakash-v |
Hey @pradnya809 @Yash-Sajwan24 |
Name: Protocol Specifications
Title: "Need automated Open API syntax validation on submitting PR to master"
Labels: C4GT Community
Description
Problem:
Sometimes when pull requests are merged, syntactical errors in the Open API document get committed on master.
Feature Request
An automated script that runs when creating a PR to the spec. It should validate transaction.yaml, meta.yaml, and registry.yaml against Open API 3.0 specification.
Goals
Expected Outcome
Acceptance Criteria
Implementation Details
Mockups / Wireframes
Project
Beckn
Organization Name:
Beckn Open Collective
Domain
Others
Tech Skills Needed:
Open API 3.0
Github actions
Node JS
Mentor(s)
Ravi Prakash
Complexity
Medium
Category
CI/CD
Sub Category
Backend, Research
The text was updated successfully, but these errors were encountered: