-
Notifications
You must be signed in to change notification settings - Fork 159
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: Updated filesystem docs with explanations for bucket URLs #1435
Docs: Updated filesystem docs with explanations for bucket URLs #1435
Conversation
✅ Deploy Preview for dlt-hub-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
||
1. You can pass the bucket URL and glob pattern or use `config.toml`. For local filesystems, use | ||
3. You can pass the bucket URL and glob pattern or use `config.toml`. For local filesystems, use | ||
`file://` or skip the schema and provide the local path in a format native for your operating system. |
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.
Just make this a bit more clear. I understand that for local filesystem, I have to use file://
or the code shown below would also work. Just make it a bit more clear.
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.
I would say you can use file://
as follows:
- code example
or
Define it as follows:
[sources.filesystem] # use [sources.readers.credentials] for the "readers" source
bucket_url='~\Documents\csv_files\'
file_glob="*"
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.
updated as per comments
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.
There is just one small change. The rest looks good
Description
Updated "verified-source/filesystem" docs for
"bucket-url"
explanation.+ used fix_grammar_gpt.py