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
Remove quotes from challenges where not needed #35481
Comments
Do we have a list of such challenges? That would help so we know when to close this issue (i.e. all have been covered). |
I do not have a list of these challenges a.t.m. In the P.R. from where this originated, @cmccormack has said something about there being other files where we had this, he might have a lead on it, otherwise a repo wide search for |
OK. |
@thecodingaviator If you do a quick search of the repo to see how many files might be affected by this, then I will query the open PRs to check all existing open PRs which might have such changes. |
@thecodingaviator @RandellDawson I would like to start from @RandellDawson I checked for |
@Nitin96Bisht we can take care of the merge conflicts if you want to submit the PR. Make sure you're working from the latest commit of the freecodecamp master branch. Please try to limit the changes in this PR to only removing the quotes so it doesn't make conflict resolution more difficult. Thanks! |
I think it's better to allow a first-time contributor to work on the issue. @Nitin96Bisht Please feel free to look through the other |
@Manish-Giri I was already working on this. I will look for |
Is this issue still on? If yes, can I help you? |
Okay No problem!! @RandellDawson |
Is this issue closed ? I am a first time contributor and would like to work on it. |
@randjanas Not yet, only the PR for English language has been merged. Check with @Nitin96Bisht for what languages he is making a PR for |
@randjanas You can start with any of the language except English language. @thecodingaviator I am not working as of now for any other language. It would be great if we allow first timer to work on this. |
hello, I would like to do this as a first timer. |
@nellybaz check with @randjanas for what languages they are working on, you can pick up the other languages and use the search and replace functionality of your code editor to make changes as discussed above |
Hi, I'm new to freecodecamp but I'm wondering if you guys still need help on this? If so, what languages should I work on? Thanks! |
Hello, I'd like to help with this issue as well. Which language folders are available? @nellybaz @randjanas Thanks! |
@LifeVirus @ageebc137 Thanks for helping out with this! AFAIK, so far only @roshni-b has claimed the Russian files. Since we haven't heard from @randjanas and @nellybaz yet, you both are welcome to pick one language each from the available four (Arabic, Chinese, Portuguese and Spanish), and start working on the corresponding files. Considering that @randjanas and @nellybaz asked first, we'll wait for them to claim the other two language versions if they're still interested. In case they (or @roshni-b) aren't, you can then continue with these remaining versions too (if you prefer). |
Thanks @Manish-Giri. I'll volunteer to do the Spanish folder. |
Thanks @Manish-Giri! I'll take the Portuguese folder! |
Hi, @Manish-Giri, after taking a cursory look in /curriculum/challenges/portuguese, and searching for |
Hi @LifeVirus. Try searching |
@ageebc137 Thank you! I removed all the instances of |
Guys, try searching for |
I made sure to search |
Do we still need help for Arabic/Chinese? @Manish-Giri @thecodingaviator? I wouldn't mind picking up those as well! |
@LifeVirus Sure! Go ahead! |
Hi @LifeVirus, are you doing both Arabic/Chinese? Do you mind if I do one of the folders? |
@ageebc137 Go for it! I'll take Chinese and you can have Arabic? |
Thanks @LifeVirus. I'll move forward with the Arabic folder. |
Hey, Would like to give a hand as its my first contribution. Where should i start? is the quote issue resolved or are there more languages where help is needed? |
@ConiWright - It appears you can choose between Spanish and Portuguese. If you have not contributed before, I suggest starting with How to Setup Free Code Camp locally and How to Work on Coding Challenges. If you have any questions, then ask in our Contributors Room on Gitter. |
@ConiWright Actually I think that all files currently are accounted for. Me and @ageebc137 have created PR's for Portuguese/Spanish, and now we're working on Chinese/Arabic. I think Russian and English were already taken before us. |
After looking for |
Hi, this will be my first contribution. I'm wondering if Chinese is still up for grab. If not, which one can I choose? Thanks. |
@UNnamed66 It appears @LifeVirus is already working on the Chinese version. |
Alright, it seems like every one of them is taken. Let me know if any one of them needs help in the future. Thanks. |
@UNnamed66 @RandellDawson, I've taken a look at the Chinese version, and it seems like the quotes are already removed. I won't be submitting a PR for it, so if you want to take a look and find something that I missed then feel free to make the PR yourself! |
Had this been fixed in the Portuguese and Russian folders? Someone needs to have a look |
@thecodingaviator Even if those languages were not translated, it is fine. We are no longer accepting translations for the current curriculum. We do not need any further PRs for the the non-English curriculum files. |
I see. Alright |
In some curriculum articles, some of the code-bits highlighted by the
<code></code>
are enclosed by a pair of quotes which are not necessary.Since the
<code>
tags already highlight code, there is no need for the quotes to be there.That is, instead of:
It should be:
One example can be found in this file
PRs:
[x] English
[x] Chinese - No instances found
[] Russian
[x] Arabic
[] Portuguese
[x] Spanish
The text was updated successfully, but these errors were encountered: