Skip to content
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

August news #375

Merged
merged 9 commits into from
Aug 26, 2022
Merged

August news #375

merged 9 commits into from
Aug 26, 2022

Conversation

maelle
Copy link
Member

@maelle maelle commented Aug 22, 2022

No description provided.

@maelle maelle requested a review from yabellini August 22, 2022 08:45
@maelle
Copy link
Member Author

maelle commented Aug 22, 2022

https://deploy-preview-375--ropensci.netlify.app/blog/2022/08/26/ropensci-news-digest-august-2022/

Copy link
Member

@yabellini yabellini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks!

maelle and others added 3 commits August 23, 2022 10:58
@mpadge
Copy link
Member

mpadge commented Aug 23, 2022

@santikka This PR is rOpenSci's upcoming News Bulletin - click the netlify link above to preview. We've announced our new pkgcheck for functions with no documented return values, and I've linked that to your recent checks as the only example we have of the checks in action. Can you please indicate whether you're okay with that link there, or whether you'd prefer us to remove it? It is a bit unfair, as you had no way of even knowing those checks had come online, so no worries at all if you'd prefer not to be linked there. Thanks!


@maelle Can you please render the .md to integrate it in the preview? @santikka You might not be able to view until rendered, but the bit i'm referring to is in index.Rmd from around Line#46.

@maelle
Copy link
Member Author

maelle commented Aug 23, 2022

@santikka
Copy link

@mpadge Yes, that is ok.

@maelle maelle marked this pull request as ready for review August 26, 2022 04:51
@maelle maelle merged commit 9dc499f into main Aug 26, 2022
@maelle maelle deleted the news-08-22 branch February 23, 2023 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants