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

Script to Generate new Schemas and update tap.py based on a Swagger Specification #20

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dluftspring
Copy link
Owner

@dluftspring dluftspring commented Dec 29, 2022

Description & motivation

It is a pain to have to update the list of available schemas when new ones are launched. This PR implements a process to pull the json schemas from hellobatons swagger specification and then generate valid json schemas and update the sub streams in tap.py using python's abstract syntax tree. The only thing left to do is author integration tests and then run this process on a cron schedule so that the available streams never deviate from what is outlined in the swagger specification.

This is a...

  • 🎉 New feature
  • 🪲 Bug fix
  • ⚙️ Refactor
  • ♻️ Housekeeping

Screenshots

Open questions

How to test

Delete or alter a schema file in schemas/ and delete the corresponding substream in tap.py then run

poetry run swagger-sync

After that the schema you deleted should be re added and it's corresponding python code should be generated as well

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.

1 participant