| We need more reviewers! |
|
We have many PRs waiting for reviews and approvals.
https://github.com/codeigniter4/CodeIgniter4/pulls If there is a PR you are interested in, please test and review it and comment on the results on GitHub. PRs not approved will not be merged. We really welcome your contributions!
Sadly, there are few active users. Personally, I can't understand everything from PR, especially if it's part of an internal system (router, uri, github actions..)
Simple CI 4 project for beginners codeigniter-expenses
If you mean that it is difficult to understand a PR with seeing the diffs on the PR page on GitHub,
yes, we may need to see whole code base to understand the changes. I recommend you install gh command. https://cli.github.com/ and you can check out the PR branch to your local environment. E.g., Code: $ gh pr checkout 7913
The code is perfectly visible in "Files changed".
The problem is precisely in understanding the changes. I see that the proposed code passes tests, is written without errors, but do I need to accept it as corrected? I can't fully understand all the logic that the code affects. Let's say we change the filters: I know that the code changes the sorting of the array or rename vars. Is she needed there? It's more about business rules than PHP
Simple CI 4 project for beginners codeigniter-expenses
(09-07-2023, 04:55 PM)kenjis Wrote: We have many PRs waiting for reviews and approvals. Hi there, I'm new to CodeIgniter and excited to start contributing. I saw your message about testing and reviewing pull requests (PRs) in the CodeIgniter repository. As a beginner, I'm eager to learn and help out. Could you please share some tips on how to get involved and understand the project better? Your guidance would be greatly appreciated!
@tarcisiodev1 Welcome!
See the docs on Contributing: https://github.com/codeigniter4/CodeIgni.../README.md especially https://github.com/codeigniter4/CodeIgni...request.md See the in-progress User Guide as references: https://codeigniter4.github.io/CodeIgniter4/ This is the User Guide of the `develop` branch. If you have any question, feel free to ask. (09-07-2023, 11:40 PM)ozornick Wrote: Let's say we change the filters: I know that the code changes the sorting of the array or rename vars. Is she needed there? It's more about business rules than PHP If you do not know what the specifications should be, you do not know if the change is correct. In rare cases, the specification is not clear. Basically, the User Guide is the specification, but it is not always detailed. In any case, if you do not know how it should behave, we will need to discuss it first. Without a common goal, it is impossible to test/review.
(09-09-2023, 04:15 PM)kenjis Wrote: @tarcisiodev1 Welcome! Hi all & @kenjis I am new to contributing on github as @tarcisiodev1 as well. I know you are all contributing a lot of time already (thank you!), but what would you think about holding an 'Onboarding Session for Newbies' via Zoom or alike for like 4 or 2 times a year? [url=https://forum.codeigniter.com/member.php?action=profile&uid=78908][/url] |
| Welcome Guest, Not a member yet? Register Sign In |
