-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Update cli.md #4329
Update cli.md #4329
Conversation
Any reason why |
Any reason why * used for creating items in markdown is reported as an error in prettier ? I don't know -- I simply run the command and check in the result npx [email protected] --write \
'{datafusion,datafusion-cli,datafusion-examples,dev,docs}/**/*.md' \
'!datafusion/CHANGELOG.md' \
README.md \
CONTRIBUTING.md |
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 @psvri -- we just need to sort out the CI error as you point out and I think this one will be good to go
Thanks for sharing the commands. Fixed the CI error in the recent commit. |
Thanks @psvri ! |
Benchmark runs are scheduled for baseline = ad5a508 and contender = e817d75. e817d75 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
Which issue does this PR close?
Closes #4239.
Rationale for this change
Updated S3 data source details in cli-usage docs.
What changes are included in this PR?
Updated the cli usage for S3 data source.
Are these changes tested?
NA
Are there any user-facing changes?
No