-
Notifications
You must be signed in to change notification settings - Fork 68
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
Add tap-msaccess
#1663
Add tap-msaccess
#1663
Conversation
✅ Deploy Preview for meltano-hub ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
430f49f
to
6f7cde0
Compare
CI failure seems to be related to |
I think so. Meltano is inject the |
Looks like there is some similar discussion in #1445 for |
tap-dbf uses the approach of nested settings, e.g. |
Co-authored-by: Edgar Ramírez Mondragón <[email protected]>
This would benefit from #1701 |
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.
Thanks @ReubenFrankel!
https://github.com/Matatika/tap-msaccess/releases/tag/v0.1.0
New take on the
tap-spreadsheets-anywhere
approach: instead of a single plugin that can connect to multiple file systems, break each out into its own plugin with defined settings, in an attempt to make configuration easier for the user.Adds 5 new plugins (possibly more in the future):
tap-msaccess
- the "base" plugin with local file system support onlytap-msaccess-azure
- Azure file system support with known settingstap-msaccess-s3
- S3 file system support with known settingstap-msaccess-http
- HTTP file system supporttap-msaccess-anywhere
- Support for (theoretically) allfsspec
backend implementationsNothing set in stone here yet, so if you have any thoughts or concerns, please raise them!