-
-
Notifications
You must be signed in to change notification settings - Fork 616
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
[v3] Add prerequisites 3/6 #984
Comments
I am willing to take this issue up (at least some of the exercises). Just let me know if I should make the PRs, one for each exercises, one for a batch of exercises or one for all of them. |
I'd suggest that you make a PR for each exercise but if you feel like you want to do a lot of exercises you can take 4-5 by PR. Trying to do all of them at once is not a good idea IMO, it's indigestible for both you and the reviewer. Idk if this might help but here are the steps I used when doing some : 1 - Doing/trying the exercise |
Good advice from @TomPradat there. I am in favor of a batch of max. 5 exercises per PR. |
Sounds good to me |
Here is an updated list (25/10/2022) :
|
Is this still open? Would love to work on it if it is! |
@safwansamsudeen Yes ! See #984 (comment) for suggestions on how to do it. Let me know if you have any questions |
@safwansamsudeen If you work on this, please start out with a small batch, e.g. 3 exercises max. |
@TomPradat Please keep this in mind https://exercism.org/blog/freeing-our-maintainers and that neither JD not me are currently actively maintaining this repo. Only encourage community contributions if you are confident that you have the time to properly review them when they come in. Otherwise it will be a frustrating experience for everyone. |
@junedev I'm new to both OSS and Exercism, so I think I'll start off with one exercise per PR. @TomPradat, I've currently opened a PR - #2159 - for the House exercise. Let me know how it goes! |
@safwansamsudeen I'll have a look next week when I have a bit more time 👍 |
@TomPradat, just a reminder about the PR. |
The idea is to split #960 into smaller chunks.
Here is the list of practices exercises to deal with for this part :
The text was updated successfully, but these errors were encountered: