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

post about rladies sp #236

Merged
merged 23 commits into from
Oct 3, 2023
Merged

Conversation

drmowinckels
Copy link
Member

No description provided.

@drmowinckels drmowinckels requested a review from a team as a code owner October 2, 2023 09:13
@drmowinckels
Copy link
Member Author

@beatrizmilz because I spent WAY too long doing this, I decided to just start it over as I completely messed up your last PR.
I'll go through this now, wait for your approval and then get it merged. I'm so sorry for all the back and forwards!

Copy link
Member Author

@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.

I think that is all from me- Mostly minor things to fix for the format.

Other than that, if you want, we are very happy to also have your Portuguese version of the post ready, if you want to share it :)

@drmowinckels drmowinckels changed the base branch from main to fork_sp_blogpost October 2, 2023 09:27
@beatrizmilz
Copy link
Member

Hi Mo @drmowinckels ! Thank you.
I will send the Portuguese version!
I would like to add a brief information at the beginning of the post, saying that it was written in April, and since then, we have received a grant from R Consortium that made it possible other things. But it will be brief

@beatrizmilz
Copy link
Member

As soon as I'm finished, I will let you know! But probably today until the end of the day, I will commit the Portuguese version. We had already written it, but I'm reviewing if there is anything that was changed, and also adding alt text in images for portuguese.

@beatrizmilz
Copy link
Member

@drmowinckels I think now is ok! Can you check if it is ok to be posted?

Copy link
Member Author

@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.

Just fixing the date left, and then we should be ok :)

@beatrizmilz
Copy link
Member

Just fixing the date left, and then we should be ok :)

Thank you Mo!

@drmowinckels drmowinckels merged commit e435f16 into rladies:fork_sp_blogpost Oct 3, 2023
2 of 3 checks passed
drmowinckels added a commit that referenced this pull request Oct 4, 2023
* add missing line

* post about rladies sp (#236)

* post about rladies sp

Co-authored-by: Beatriz Milz <[email protected]>
Co-authored-by: Beatriz Milz <[email protected]>
Copy link
Contributor

@SoyAndrea SoyAndrea left a comment

Choose a reason for hiding this comment

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

everything is fine for me, beautiful post

@beatrizmilz
Copy link
Member

everything is fine for me, beautiful post

Hi Andrea! The post was merged. https://rladies.org/blog/2023-06-05-increasing-diversity-sao-paulo/
It would be nice to share in social media!

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.

3 participants