-
Notifications
You must be signed in to change notification settings - Fork 18
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: small README improvements #57
Conversation
Welcome to Codecov 🎉Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment. Thanks for integrating Codecov - We've got you covered ☂️ |
Thanks, looks good. I wonder if installation instructions should be at the beginning or at the end. The pak readme has them at the end, but would we need a ToC to support this? No strong preference. I moved the |
The README has a ToC by default on GitHub anyway, however I am more used to seeing installation instructions at the beginning. 🤔
I wanted the two examples to be self-contained as it means only one copy-paste to try them out. 😁 Maybe you can add a chunk with echo=FALSE that loads the library? |
The examples are no longer self-contained because plain text separates them, you'll copy individual pieces anyway. I wonder if we can host the full examples in standalone build-ignored R scripts (with I don't see the ToC in my GitHub. |
I can't make any edit now, but regarding the TOC, if you click on the "three lines" icon near "README.md" you'll get a TOC. https://github.blog/changelog/2021-04-13-table-of-contents-support-in-markdown-files/ Now if even you don't know about this, many users won't know either. |
Thanks! |
Fix #34
Other tweaks