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 confusing Palindrome example #12

Closed
wants to merge 1 commit into from

Conversation

estsauver
Copy link

This example used palindromes to describe the reverse_each method, which seemed confusing and less helpful than an example that showed how the array reversed the order of the elements.

This example used palindromes to describe the reverse_each method, which seemed confusing and less helpful than an example that showed how the array reversed the order of the elements.
@zzak
Copy link

zzak commented Jul 11, 2013

@estsauver Thank you for your contribution, good find!

This was resolved by ruby/ruby@be698db, sorry for the delay I must have missed your email!

@zzak zzak closed this Jul 11, 2013
@estsauver
Copy link
Author

Hey no problem! I wasn't sure how attentively this repo was watched. Thanks for merging this.

@zzak
Copy link

zzak commented Jul 12, 2013

@estsauver I am the current maintainer, so if you have any questions feel free to ping me!

HonoreDB pushed a commit to HonoreDB/ruby that referenced this pull request Nov 12, 2013
  Patch by Earl St Sauver [Fixes documenting-ruby/ruby-12]
  documenting-ruby#12


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@41917 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
tenderlove pushed a commit to tenderlove/ruby that referenced this pull request Jan 24, 2014
  Patch by Earl St Sauver [Fixes documenting-ruby/ruby-12]
  documenting-ruby#12


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@41917 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
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

2 participants