chore(challenges): fixed a minor spelling error

freeCodeCamp / curriculum

The freeCodeCamp curriculum and lesson editor

Creative Commons Attribution Share Alike 4.0 International

81 stars 124 forks source link

chore(challenges): fixed a minor spelling error #233

Closed lauren-van-sloun closed 6 years ago

lauren-van-sloun commented 6 years ago

Description

Fixed a minor spelling error ("a option" => "an option")

Pre-Submission Checklist

[x] Your pull request targets the dev branch.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/challenge-tests)
[x] All new and existing tests pass the command npm test.
[x] Use npm run commit to generate a conventional commit message. Learn more here: https://conventionalcommits.org/#why-use-conventional-commits
[x] The changes were done locally on your machine and NOT GitHub web interface. If they were done on the web interface you have ensured that you are creating conventional commit messages.

Checklist:

[x] Tested changes locally.
[x] Addressed currently open issue #17990

Closes #17990

moT01 commented 6 years ago

@Ashera138 looks good I think, that's my first time merging something so hopefully I did it correctly - You should maybe put in the description of the PR that you also changed all that spacing so the person reviewing it knows.

lauren-van-sloun commented 6 years ago

I actually noticed the extra white spaces after pushing (oops). I think maybe it was because I did the edit in Notepad++? I'm not sure.

Thanks again for helping me on my first pr! 👍

moT01 commented 6 years ago

Perhaps, you didn't deliberately add them I take? - I was going to say they should be deleted, but I noticed several other files shared that structure so I didn't think it was a problem. Thanks for contributing 👍

lauren-van-sloun commented 6 years ago

Yeah, I didn't intentionally add them. And you're welcome. 😄

scissorsneedfoodtoo commented 6 years ago

@Ashera138, thanks again for your contribution! And thank you for taking over the QA and merge @moT01 :+1:

No need to worry about all of the extra line breaks. That's just the new prettier format that runs with the linter. Really threw me the first time, too!

lauren-van-sloun commented 6 years ago

Oh alright, good to know. Thanks for mentioning that @scissorsneedfoodtoo. And thanks you guys for sharing an easy issue for a beginner to grab. 😃

raisedadead commented 5 years ago

:tada: This PR is included in version 3.2.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: