Skip to content

[en,ja]Add regression-of-hash-reject-in-ruby-2-1-1 #666

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

Merged
merged 2 commits into from
Mar 11, 2014

Conversation

sorah
Copy link
Member

@sorah sorah commented Mar 10, 2014

Solves #665

This is requested by @nurse and @hsbt

CC: @zzak

@makimoto
Copy link
Contributor

IMO, we should add a notice about it to 2.1.1 release announcement. Thought?

@sorah
Copy link
Member Author

sorah commented Mar 10, 2014

Agreed: 1d35e76

@@ -0,0 +1,36 @@
---
layout: news_post
title: "Ruby 2.1.1 に含まれる Hash#reject のデグレードについて"
Copy link
Contributor

Choose a reason for hiding this comment

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

How about using "リグレッション" (or "不具合" simply) instead of "デグレード"?
In the English version, a word "regression" is used for it.

Copy link
Member Author

Choose a reason for hiding this comment

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

"不具合" !!!!

@hsbt
Copy link
Member

hsbt commented Mar 10, 2014

@sorah Good work, but I have no time to review your issue 🙏

@sorah
Copy link
Member Author

sorah commented Mar 10, 2014

@hsbt thank, and I see that you have no time

I want to merge this after @nurse's approval

@@ -16,6 +16,8 @@ lang: ja

なお以前お伝えしましたとおり、Ruby 2.1 では[新しいバージョン表記](https://www.ruby-lang.org/ja/news/2013/12/21/semantic-versioning-after-2-1-0/)を採用していますので、単純に「Ruby 2.1.1」というのがこのリリースの名称になります。

__追記:__ Ruby 2.1.1 において、`Hash#reject` のデグレードを発見しています。詳細はこちらをご覧ください: https://www.ruby-lang.org/ja/news/2014/03/10/regression-of-hash-reject-in-ruby-2-1-1
Copy link
Contributor

Choose a reason for hiding this comment

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

  1. A word "デグレード" is now replaced with "不具合" in the article about the regression.
  2. "...を発見しています" is a little unnatural. How about "が発見されました" or "を発見しました"?

@makimoto
Copy link
Contributor

The Japanese contents look good to me. 👍
(You may want to squash your commits.)

In Ruby 2.1.0 or earlier, the `reject` method in any class that inherits `Hash` returns an object of its own class.
But in Ruby 2.1.1, this behavior has changed accidentally to return always a plain Hash object, not of the inherited class.

```
Copy link
Contributor

Choose a reason for hiding this comment

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

highlighted code with

{% highlight ruby %}
... code ...
{% endhighlight %}

@nurse
Copy link
Member

nurse commented Mar 11, 2014

OK, thanks

sorah added a commit that referenced this pull request Mar 11, 2014
[en,ja]Add regression-of-hash-reject-in-ruby-2-1-1
@sorah sorah merged commit f489580 into ruby:master Mar 11, 2014
@sorah sorah deleted the ruby210-hash-reject-regression branch March 11, 2014 04:23
@zzak
Copy link
Member

zzak commented Mar 11, 2014

Good work everyone!

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.

6 participants