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

Static analysis on schemas #60

Open
2 tasks
rafaparadela opened this issue Jan 10, 2019 · 2 comments
Open
2 tasks

Static analysis on schemas #60

rafaparadela opened this issue Jan 10, 2019 · 2 comments
Projects

Comments

@rafaparadela
Copy link
Member

We should be able to perform static analyisis on schemas. The idea is that when uploading that schema to Catamorph, we should be able to warn the user if they're doing something that we know is wrong, or hint a different approach.

For this issue to be done, we need to:

  • Create all the plumbing needed to create rules
  • Define and implement rules
@pepegar pepegar added this to To do in skeuomorph Jul 5, 2019
@diesalbla
Copy link

@rafaparadela Taking a quick look at some issues, I was wondering if you could give an example of a kind of error, or warning, that you had in mind when creating the issue. One example I can think of, which we have discussed, is to have a reference checker, that makes sure that whenever you have a reference to a type, that type is defined in the file. This is common in OpenApi, with the components section. Is that something you were thinking about?

@rafaparadela
Copy link
Member Author

rafaparadela commented Jul 16, 2019

This issue was originally created in Marlow, and I moved it to here. So I would say that @pepegar is the original reporter.
I guess that the main goal for this issue is providing a mechanism to warn when some the schemas don't fulfill the rules defined by the own serialization system. For instance, incompatible naming, wrong number of fields, empty coproducts, idk, stuff like that. If we set a rules-engine to validate the schema before operating with it, then it would provide like more security.
@pepegar correctly if I'm wrong, please.

skeuomorph automation moved this from To do to Done Jul 16, 2019
@rafaparadela rafaparadela reopened this Jul 16, 2019
skeuomorph automation moved this from Done to In progress Jul 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
skeuomorph
  
In progress
Development

No branches or pull requests

2 participants