-
Notifications
You must be signed in to change notification settings - Fork 14
De-duplicate README content and update installation instructions #274
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
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #274 +/- ##
=======================================
Coverage 96.28% 96.28%
=======================================
Files 32 32
Lines 3606 3606
=======================================
Hits 3472 3472
Misses 134 134 ☔ View full report in Codecov by Sentry. |
jasonmcewen
left a comment
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.
LGTM! Please merge at your convenience.
Resolves #264
Resolves #222
README.mdto make it more portable (uses absolute rather than relative links, avoids GitHub Flavour Markdown emoji references in favour of Unicode characters, adjusts some formatting and styling) to allow content to be reused..pip_readme.rstin favour of just using mainREADME.mdand updatesdocs/index.rstto includeREADME.mdcontent.README.mdupdates to reflect current approach for installing (optional) dependencies, and old installation instructions indocs/user_guide/install.rstremoved.