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

Review 2 of Softwarereview_reviewer.pt.Rmd-pt-auto #853

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

samuel-carleial
Copy link
Contributor

@pedropark99
Here is my R2 to this task. I had a detailed look on the english version to the side, and tried to improve the PT-BR translation. Espero que tenha melhorado.

@samuel-carleial
Copy link
Contributor Author

Should the PR be actually to the working branch? Did I do something wrong here?

@pedropark99
Copy link
Contributor

@samuel-carleial Yes! This PR should aim to the working branch, instead of targeting the main branch.

Therefore, the target branch should be ropensci:softwarereview_reviewer.pt.Rmd-pt-auto

@samuel-carleial
Copy link
Contributor Author

samuel-carleial commented Nov 26, 2024 via email

@yabellini
Copy link
Member

@samuel-carleial Here are instructions on how to change the target branch: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/changing-the-base-branch-of-a-pull-request

I can also make the change; let me know. Thank you for the review!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants