Skip to content

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

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

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

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