-
Notifications
You must be signed in to change notification settings - Fork 111
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
Status review for unfinished articles #416
Comments
@Peruibeloko I agree but we probably need to be sure that Untouched topics have no PR, so three months might also be a good waiting period. Thanks. |
@odsantos most of the Untouched topics are months or years out of date with master. I can go over them and append the age of the assignment if you'd like |
@Peruibeloko no, there is no need, as long as no work is present this seems good. Thanks. |
Glad to help, let me know if you need anything! |
I will arrange it as soon as possible, I had forgotten to translate, thank you for reminding me. |
@HakaCode thanks! Just a quick reminder to keep the PR contained to the specific as article you're translating! |
@Peruibeloko @odsantos I think now I just wait for it to be reviewed and accepted. |
Hey guys! About the Extending built-in classes article, I'm now just waiting a review in the #413 PR I also translated these two articles below, but they are unchecked even thought I created their pull request with their article titles:
How can I mark them as checked? |
@danilolmc I didn't include your class extension article since I remember seeing you picking it one of these days. Marking them as done is probably restricted to either the bot or @iliakan |
Great, thank you! @Peruibeloko |
Instead of reopening the PR, we could contribute directly to it (PR number), and if the user's branch or repository is protected, then the task goes on to be reopened in a new PR, what do you think? In reviews, I'm looking to contribute by adding commits directly to the user's forked repository that is associated with the PR, instead of just registering my comments for simple things, trying to speed up the merge process, instead of waiting for the user's response that may have abandoned the PR |
My deduction, the bot apparently did not act on the PRs because the merger was apparently carried out without following the necessary review steps (review and approval process) and perhaps because it failed to mark the PR's hash on the list due to absences in the approval and registration procedures of the task on behalf of the person.
Maintainers have the permission to update PR #1. I have updated the list considering the PRs have been completed. |
I agree with 03 months instead of 06, we need agility and evolution of this repository instead of waiting for PRs that haven't even been touched yet. |
Hello everyone, I would really like to contribute to this repository, but I'm kind of new to open source projects, I saw that there are a lot of PRs open and after this list that @Peruibeloko sent, it gave me a little idea, but if I want to contribute , is there anyone active I can talk to? Note: Or I just raise a Pr and who knows, one day it will be approved. |
Hey @savionascimentodev ! You can take a look at #1 issue. Hope it can help you... |
@savionascimentodev Seja bem-vindo mano! O processo é bem simples:
Esse é o resumo executivo, posso te auxiliar lá no #1 diretamente, já que aqui é mais voltado para outro assunto :D |
Vou fechar essa issue, já que conseguimos trazer atenção ao problema e movimentamos o #1 |
I went over the translation checklist and classified each item in one of three categories:
And this is the current status:
We can solve some of these problems as such:
What does everyone think?
The text was updated successfully, but these errors were encountered: