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

Posit conf 2023 summary #235

Merged
merged 32 commits into from
Oct 10, 2023
Merged

Posit conf 2023 summary #235

merged 32 commits into from
Oct 10, 2023

Conversation

shannonpileggi
Copy link
Contributor

All reviews are welcome! Here is a quick first draft, I am sure it needs edits.

@github-actions
Copy link

github-actions bot commented Oct 2, 2023

🎉 The preview is built! Check it out

Copy link
Member

@drmowinckels drmowinckels left a comment

Choose a reason for hiding this comment

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

Here are just some simple first edits from me. Ill take a closer look later :)

shannonpileggi and others added 20 commits October 2, 2023 10:11
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
…eception-at-posit-conf/index.en.md

Co-authored-by: Athanasia Monika Mowinckel <[email protected]>
@shannonpileggi
Copy link
Contributor Author

@drmowinckels i updated folder name and date but now i'm not seeing the news post in the preview and not seeing what i did wrong 🤔

@shannonpileggi shannonpileggi self-assigned this Oct 8, 2023
@yabellini
Copy link
Member

I added two more suggestion to mention the RConsortium in the right way. Please, check it is ok with the grammar. Thank you so much!!

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

🎉 The preview is built! Check it out

Copy link
Contributor

@sayantikabanik sayantikabanik left a comment

Choose a reason for hiding this comment

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

Looks fantastic, had few minor tweaks in mind which I believe are addressed by other reviewers :)

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

🎉 The preview is built! Check it out

@shannonpileggi
Copy link
Contributor Author

I added two more suggestion to mention the RConsortium in the right way. Please, check it is ok with the grammar. Thank you so much!!

thanks, @yabellini! i tweaked the wording slightly.

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

🎉 The preview is built! Check it out

@github-actions
Copy link

github-actions bot commented Oct 9, 2023

🎉 The preview is built! Check it out

@shannonpileggi
Copy link
Contributor Author

ok all - i made one more comment. then we need someone to approve before we publish. thanks, everyone!

@github-actions
Copy link

🎉 The preview is built! Check it out

@drmowinckels drmowinckels merged commit bde7216 into main Oct 10, 2023
3 checks passed
@drmowinckels drmowinckels deleted the posit-conf-2023-summary branch October 10, 2023 13:45
@sheilasaia
Copy link

@shannonpileggi, two last comments if it's not too late! (1) can you include a link to the r-ladies discussion board on the name change in that section? (2) I saw one instance of "RLadies" and another of "R Ladies" but they should be "R-Ladies", I think.

@cosimameyer
Copy link
Contributor

I think it’s already merged but we can open a new branch and do a fix in a new PR, @sheilasaia ☺️

@drmowinckels
Copy link
Member

The discussion has yet to be properly opened, so we cant really link to it yet, @sheilasaia . But the other parts are good catch!

@drmowinckels
Copy link
Member

fixed in #249

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.

10 participants