-
Notifications
You must be signed in to change notification settings - Fork 1
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
docs: standard for building new connector #49
Comments
@pnadolny13 this is a great start - let's just get a PR up and ready, perhaps within MeltanoLabs? The other item(s) I would add is:
|
@pnadolny13 Should probably clarify this means integration tests |
pnadolny13
added a commit
that referenced
this issue
May 31, 2023
Closes #49 - cleanup the README and add a link out to the connector dev standards guide - add connector dev standards guide Let me know if I missed any tips or if you think theres better formatting or location for this. --------- Co-authored-by: Edgar R. M. <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We should document the minimum standard we have for creating a new connector. This can act as a check list for us to know that a connector has completed initial development and should be made available to the community. It also acts as a way to communicate with external contributors or contractors what the expectation and best practices are for creating a new connector.
In #37 (comment) I created a list of common checklist steps, this could be a good starting place.
main
Some other best practices that have been mentioned are:
--about --format=markdown
output@kgpayne @edgarrmondragon @tayloramurphy let me know what you think of these and if you have more. We can add them to the README or a contributors guide when we're ready.
The text was updated successfully, but these errors were encountered: