-
Notifications
You must be signed in to change notification settings - Fork 59
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
base: main
Are you sure you want to change the base?
Review 2 of Softwarereview_reviewer.pt.Rmd-pt-auto #853
Conversation
Should the PR be actually to the working branch? Did I do something wrong here? |
@samuel-carleial Yes! This PR should aim to the working branch, instead of targeting the main branch. Therefore, the target branch should be |
Ok eu vou tentar fazer direto depois. Obrigado pela resposta
…On Tue, 26 Nov 2024, 21:57 Pedro Faria, ***@***.***> wrote:
@samuel-carleial <https://github.com/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
—
Reply to this email directly, view it on GitHub
<#853 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJ4V64NDNSKIITCUEWHYAST2CTOFFAVCNFSM6AAAAABSRKK4EOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBRHEZDCMJUGY>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@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! |
@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.