Skip to content
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

Closed
thecodingaviator opened this issue Mar 1, 2019 · 42 comments · Fixed by #35772
Closed

Remove quotes from challenges where not needed #35481

thecodingaviator opened this issue Mar 1, 2019 · 42 comments · Fixed by #35772
Labels
scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.

Comments

@thecodingaviator
Copy link
Contributor

thecodingaviator commented Mar 1, 2019

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:

"<code>stuff</code>"

It should be:

<code>stuff</code>

One example can be found in this file

PRs:

[x] English
[x] Chinese - No instances found
[] Russian
[x] Arabic
[] Portuguese
[x] Spanish

@thecodingaviator thecodingaviator added scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory. first timers only Open for anyone doing contributions for first time. You do not need permission to work on these. labels Mar 1, 2019
@RandellDawson
Copy link
Member

RandellDawson commented Mar 1, 2019

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).

@thecodingaviator
Copy link
Contributor Author

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 "<code> can be run

@RandellDawson
Copy link
Member

OK.

@RandellDawson
Copy link
Member

@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.

@Nitin96Bisht
Copy link
Contributor

Nitin96Bisht commented Mar 2, 2019

@thecodingaviator @RandellDawson I would like to start from /curriculum/challenges/english. Kindly let me know May I start now to avoid file merge conflict.

@RandellDawson I checked for /curriculum/challenges/english. There are 18 files that will be affected.

@cmccormack
Copy link
Member

@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!

@Manish-Giri
Copy link
Contributor

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 help wanted issues. There are many of them we could use your help on!
In case this one is still not picked up in a few days, you can then work on this.

@Nitin96Bisht
Copy link
Contributor

Nitin96Bisht commented Mar 2, 2019

@Manish-Giri I was already working on this. I will look for help wanted issues also from now onwards. 👍

@Sumer16
Copy link
Contributor

Sumer16 commented Mar 12, 2019

Is this issue still on? If yes, can I help you?

@RandellDawson
Copy link
Member

@Sumer16 This one already has an open PR (see above #35493).

@Sumer16
Copy link
Contributor

Sumer16 commented Mar 13, 2019

Okay No problem!! @RandellDawson

@randjanas
Copy link

Is this issue closed ? I am a first time contributor and would like to work on it.

@thecodingaviator
Copy link
Contributor Author

@randjanas Not yet, only the PR for English language has been merged. Check with @Nitin96Bisht for what languages he is making a PR for

@Nitin96Bisht
Copy link
Contributor

@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.

@nellybaz
Copy link

hello, I would like to do this as a first timer.
Where should I start?

@thecodingaviator
Copy link
Contributor Author

@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

@Manish-Giri Manish-Giri added the help wanted Open for all. You do not need permission to work on these. label Mar 29, 2019
@LifeVirus
Copy link
Contributor

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!

@ageebc137
Copy link
Contributor

Hello, I'd like to help with this issue as well. Which language folders are available? @nellybaz @randjanas Thanks!

@Manish-Giri
Copy link
Contributor

Manish-Giri commented Apr 5, 2019

@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).

@ageebc137
Copy link
Contributor

Thanks @Manish-Giri. I'll volunteer to do the Spanish folder.

@LifeVirus
Copy link
Contributor

Thanks @Manish-Giri! I'll take the Portuguese folder!

@LifeVirus
Copy link
Contributor

LifeVirus commented Apr 5, 2019

Hi, @Manish-Giri, after taking a cursory look in /curriculum/challenges/portuguese, and searching for
"<code> stuff </code>"", and &quot, I haven't found any examples of "<code> stuff </code>"", and the only places where "<code> or </code>" pop up are in Tests, and in assert fields, where it's unrelated I think. Am I missing something? Sorry for the bother!

@ageebc137
Copy link
Contributor

ageebc137 commented Apr 5, 2019

Hi @LifeVirus. Try searching </code> &quot; making sure there is a space in between. I noticed there were a lot of false positives when I did my search. When I searched, I was able to find &quot; <code>...</code> &quot; in the Portuguese folder at declare-the-doctype-of-an-html-document.portuguese.md.

@LifeVirus
Copy link
Contributor

@ageebc137 Thank you! I removed all the instances of &quot; <code>…</code> &quot;, and I think that that may be all of the instances in the Portuguese files. Would it be possible for me to work on the other untaken languages as well, or should I wait on @randjanas and @nellybaz to see if they are picking up Arabic/Chinese?

@thecodingaviator
Copy link
Contributor Author

Guys, try searching for "<code> and </code>" separately

@ageebc137
Copy link
Contributor

I made sure to search "<code>, </code>", " <code>, </code> ", &quot; <code>, &quot;<code>, </code>&quot;, and </code> &quot;, all separately.

@LifeVirus
Copy link
Contributor

Do we still need help for Arabic/Chinese? @Manish-Giri @thecodingaviator? I wouldn't mind picking up those as well!

@Manish-Giri
Copy link
Contributor

@LifeVirus Sure! Go ahead!

@ageebc137
Copy link
Contributor

Hi @LifeVirus, are you doing both Arabic/Chinese? Do you mind if I do one of the folders?

@LifeVirus
Copy link
Contributor

@ageebc137 Go for it! I'll take Chinese and you can have Arabic?

@ageebc137
Copy link
Contributor

Thanks @LifeVirus. I'll move forward with the Arabic folder.

@DannyBoris
Copy link

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?
Thanks in advance

@RandellDawson
Copy link
Member

@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.

@LifeVirus
Copy link
Contributor

@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.

@LifeVirus
Copy link
Contributor

After looking for "<code>, </code>", " <code>, </code> ", &quot; <code>, </code> &quot;, &quot;<code>, and </code>&quot;, I couldn't find any instances of any of these in the Chinese folder. Somebody else can verify if they wish, but as far as I've searched, I can't find anything in the Chinese folder specifically, even though they exist in my local (non-fixed) Arabic, Portuguese, Spanish, etc. folders, so I don't think a PR needs to be made for the Chinese folder specifically.

@UNnamed66
Copy link

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.

@RandellDawson
Copy link
Member

@UNnamed66 It appears @LifeVirus is already working on the Chinese version.

@UNnamed66
Copy link

@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.

@LifeVirus
Copy link
Contributor

@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!

@thecodingaviator
Copy link
Contributor Author

Had this been fixed in the Portuguese and Russian folders? Someone needs to have a look

@thecodingaviator thecodingaviator added status: in progress and removed first timers only Open for anyone doing contributions for first time. You do not need permission to work on these. help wanted Open for all. You do not need permission to work on these. labels May 7, 2019
@RandellDawson
Copy link
Member

@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.

@thecodingaviator
Copy link
Contributor Author

I see. Alright

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
scope: curriculum Lessons, Challenges, Projects and other Curricular Content in curriculum directory.
Projects
None yet