-
Notifications
You must be signed in to change notification settings - Fork 451
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
feat: providers package #3308
feat: providers package #3308
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small nit comment made inline.
💯 to having it in its own package.
Can you be sure to ping me when this moves permanently? I have a couple of scripts that are watching the providers.yaml for changes to take some actions on the new provider that was added. Thanks! |
Yes probably not before last week of january, until then nothing changes |
Changes
Contributes to https://linear.app/nango/issue/NAN-2265/remove-shared-from-cli
Split of #3297
@nangohq/providers
Only contains providers.yaml. Compiled and included in tooling but not used in code yet