-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
various documentation updates #3005
Conversation
fix some links and docs fix docker build for datafusion-cli and update docs improve left nav link naming for clarity consolidated the documentation for the CLI into one page per issue apache#1352 fix the CSV schema inference in datafusion-cli docs per apache#3001
Codecov Report
@@ Coverage Diff @@
## master #3005 +/- ##
========================================
Coverage 85.84% 85.85%
========================================
Files 283 289 +6
Lines 51658 51890 +232
========================================
+ Hits 44347 44550 +203
- Misses 7311 7340 +29
📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more |
remove duplicate example usage, instead directing user to other page for examples add reference to talk/slides from summit
Updated per review. I'm not done with this. Still working on organizing the content and creating new content. |
@kmitchener What do you think about breaking some of these changes out into smaller PRs? For example, the formatting changes to the roadmap could be reviewed as a standalone change. This might make it easier to review the more substantive changes here. |
@andygrove sure, yes, I can do that. I got a little exploratory and just kept making changes, unsure how they would work out. I'll submit separate PR for the simple cleanups and see how I can break this up further for easier review. I've been away from this for a few days, maybe fresh eyes will help. |
closing this PR as it was exploratory and too big and all it's changes are going in other PRs |
Which issue does this PR close?
Closes #1352
Closes #3001
Closes #2866
You can also close #2558 and close #1813 though they're not directly related to this PR
Rationale for this change
What changes are included in this PR?
Are there any user-facing changes?
The docs are slightly better.