Check if required parameters were added

[Leandro de Oliveira]

Hi, I’d like to implement a check for adding a required parameter to a spec because that’s a breaking change but I’m struggling to implement it, would you be so kind enough to finish it and incorporate it in your project? Here is the branch: https://bitbucket.org/lehphyro/openapi-diff/branch/required-query-param-check

[Ben Sayers]

Hi @Leandro de Oliveira ,

Thanks for making a start on this! I’ll add this to my teams backlog to review and finish. I can’t make any promises about how quickly we can get this into master but we’ll try to get it done as soon as we can.

[Leandro de Oliveira]

Great! Please let me know if you need me to create a PR or give you access to my forked project.

[Ben Sayers]

@Leandro de Oliveira in order for us to include your contributions we will need a signed Contributor License Agreement. You can read more details and find links in our contributing docs. If you are happy to agree to this please follow the process to sign it and let me know when you are done. If you would prefer not to agree to this, we are happy to add this feature to the tool, but will need to implement it from scratch.

[Leandro de Oliveira]

Hi Ben, I just signed the CLA as an individual.

[Ben Sayers]

Thanks @Leandro de Oliveira , I can confirm it’s been received.

[Martijn Vegter]

Hi @Ben Sayers , any updates regarding this issue?

[Lorenz HENK]

Hey, any update on this? Would be highly appreciated

[Jean-François Côté]

@Ben Sayers Any update on this? This is preventing me from using this solution because changing an existing parameter to required or to add a required parameter is a breaking change that is not detected by this project.