Skip to content
This repository has been archived by the owner on Jan 30, 2023. It is now read-only.

remove @expectedException, etc measure #29

Merged
merged 1 commit into from Jul 11, 2020
Merged

remove @expectedException, etc measure #29

merged 1 commit into from Jul 11, 2020

Conversation

tarohida
Copy link
Contributor

As you know, "@ExpectedException", etc annotation was deprecated in phpunit 8.x, and removed in 9.x.
ref: gordalina/cachetool#141

I synced Japanese document to English one.
https://github.com/sebastianbergmann/phpunit-documentation-english/blob/master/src/writing-tests-for-phpunit.rst#testing-exceptions

I checked this work. It seems ok.

Run make html to make sure that the documentation can be built and looks as expected

@tarohida
Copy link
Contributor Author

tarohida commented May 28, 2020

#26 で言及されてましたね。
あと、コラボレータの方は日本の方なんですね。よろしくお願いします。

9.x系で無効となった「@ExpectedException」等のannotationについて記載されたままだったので、その記載を削除しました。内容として整合性はとれていると思います。

全部訳すのは大変だったので、一部のみの追随となります。
もしまとめての作業が進行中だったりと私の部分的な変更に差し障りがある場合は、declineしていただいて構いません。
よろしくお願いします。

@kuromoka
Copy link
Contributor

@taro-hida
ご連絡遅くなり申し訳ございません!
確かに英語版見ても削除されているので消したほうがよさそうですね。
https://phpunit.readthedocs.io/en/9.2/writing-tests-for-phpunit.html

マージしておきます!

@kuromoka kuromoka merged commit 8cc0737 into sebastianbergmann:master Jul 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants