-
-
Notifications
You must be signed in to change notification settings - Fork 132
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
provide recommendation for migration to R 4.0 #830
Conversation
Deploy preview for stupefied-einstein-5bde90 ready! Built with commit 34d3071 https://deploy-preview-830--stupefied-einstein-5bde90.netlify.app |
Looks great to me. Thanks for setting this up! |
I'm adding an example from the R Social Science lesson to make sure that there is a positive recommendation for lesson maintainers to draw from.
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.
Small typo.
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.
Excellent ^_^
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.
I've made changes to the history of factors and suggestions for ways maintainers can more meaningfully introduce them in their lessons.
Co-authored-by: Allison Theobold <atheobol@calpoly.edu>
Co-authored-by: Allison Theobold <atheobol@calpoly.edu>
Thank you for your input @atheobold! This definitely improves the tone of the post and gives valuable information. Would you mind adding your name to the Author list? https://github.com/carpentries/carpentries.org/pull/830/files#diff-97b5acc7d7fc4a2fc596284eee1ad44dR3 This post will go live tomorrow at 1400 UTC. |
Co-authored-by: Allison Theobold <atheobol@calpoly.edu>
- updated blog post tags - added the summary recommendation as a standalone quote for emphasis - reworded two sections for clarity - removed the dissonance quantifier ' small ' from a sentence
Derived from conversations in the June maintainer meeting.
cc @jcoliver @HaoZeke