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

Revise description for Concatenate Arrays with .concat Challenge #11855

Merged
merged 1 commit into from Nov 29, 2016

Conversation

johnnyqbui
Copy link
Contributor

@johnnyqbui johnnyqbui commented Nov 29, 2016

Pre-Submission Checklist

  • Your pull request targets the staging branch of FreeCodeCamp.
  • Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
  • You have only one commit (if not, squash them into one commit).
  • All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

  • Small bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds new functionality)
  • Breaking change (fix or feature that would change existing functionality)
  • Add new translation (feature adding new translations)

Checklist:

Description

Revised description for Concatenate Arrays with .concat challenge for a more accurate description of .concat.


Fixed checklist by @abhisekp - 29th Nov. 2016 - 8:33 AM IST

@BerkeleyTrue BerkeleyTrue added the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Nov 29, 2016
Copy link

@t3h2mas t3h2mas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. IMO array(s) is better left as arrays, since two is the minimum for concatination

@abhisekp
Copy link
Member

abhisekp commented Nov 29, 2016

@johnnyqbui How does the challenge description look like after this modification? Any screenshot?

@abhisekp abhisekp changed the title revised description for Concatenate Arrays with .concat Challenge Revise description for Concatenate Arrays with .concat Challenge Nov 29, 2016
@johnnyqbui
Copy link
Contributor Author

@t3h2mas ok, i've changed it to arrays
@abhisekp
capture

What do you guys think?

@camperbot
Copy link
Contributor

@johnnyqbui updated the pull request.

"Here are examples of <code>concat</code> being used to concatenate arrays:",
"Example 1: <code>otherArray</code> onto the end of <code>oldArray</code>:",
"<blockquote>newArray = oldArray.concat(otherArray);</blockquote>",
"Example 2: <code>newArray</code> with all values of <code>arr1, arr2,</code> and <code>arr3</code>:",

This comment was marked as off-topic.

@camperbot
Copy link
Contributor

@johnnyqbui updated the pull request.

"Here are examples of <code>concat</code> being used to concatenate arrays:",
"Example 1: <code>otherArray</code> onto the end of <code>oldArray</code>:",
"<blockquote>newArray = oldArray.concat(otherArray);</blockquote>",
"Example 2: <code>newArray</code> with all elements of <code>arr1, arr2,</code> and <code>arr3</code>:",

This comment was marked as off-topic.

@camperbot
Copy link
Contributor

@johnnyqbui updated the pull request.

@camperbot
Copy link
Contributor

@johnnyqbui updated the pull request.

Copy link
Member

@abhisekp abhisekp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✨ 😸 👍 LGTM 😃 👍 ✨

@johnnyqbui
Copy link
Contributor Author

@abhisekp Yay! my first approved PR, thanks!

@abhisekp abhisekp merged commit 2434f26 into freeCodeCamp:staging Nov 29, 2016
@BerkeleyTrue BerkeleyTrue removed the status: waiting review To be applied to PR's that are ready for QA, especially when additional review is pending. label Nov 29, 2016
@abhisekp
Copy link
Member

Thanks for this one. 🎉 😃

Let it keep coming. Love your work 💗

You might want to check out Writing Good Git Commit Messages

Enjoy! 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants