-
Notifications
You must be signed in to change notification settings - Fork 14.6k
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 README.md #9731
Update README.md #9731
Conversation
Update and reorganize README to make it easier to navigate and use for community visitors.
We would appreciate it if you could provide us with more info about this issue/pr! Please do not leave the |
Codecov Report
@@ Coverage Diff @@
## master #9731 +/- ##
==========================================
+ Coverage 70.49% 70.51% +0.02%
==========================================
Files 402 402
Lines 12564 12564
Branches 3112 3112
==========================================
+ Hits 8857 8860 +3
+ Misses 3593 3590 -3
Partials 114 114
Continue to review full report at Codecov.
|
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.
Overall LGTM, good improvements, but let's get rid of the horizontal bars.
See it rendered here in your branch: https://github.com/ceohockey60/incubator-superset/tree/patch-1
Update and reorganize README to make it easier to navigate and use for community visitors.
CATEGORY
Choose one
SUMMARY
Update and reorganize the README with better navigation and headline anchors to make it easier to navigate and use for community visitors. Also updated most Superset company user with the github handle of the person who filed the initial PR to give proper recognition.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TEST PLAN
ADDITIONAL INFORMATION
REVIEWERS