Skip to content

Conversation

@stevedlawrence
Copy link
Contributor

With recent versions of javadoc, checks are made to ensure that throws
clasues exist for any javadoc @throws tags. This patch converts Scala
@throws annotations to the Java throws clause so that these checks can
pass.

With recent versions of javadoc, checks are made to ensure that throws
clasues exist for any javadoc @throws tags. This patch converts Scala
@throws annotations to the Java throws clause so that these checks can
pass.
@ktoso
Copy link
Contributor

ktoso commented Jun 22, 2015

Looks good but I'm not very deep into this codebase, second review by @rkuhn or @2m (or @bantonsson)?

@ktoso
Copy link
Contributor

ktoso commented Jun 22, 2015

In order to merge this we'll have to ask you to sign out contributor agreement, it's pretty easy: http://www.typesafe.com/contribute/cla just follow the steps on that site and confirm with yout github account (no physical paperwork needed).

@bantonsson
Copy link
Contributor

LGTM

@stevedlawrence
Copy link
Contributor Author

I have signed the contributor agreement.

@ktoso
Copy link
Contributor

ktoso commented Jun 22, 2015

Great, thanks @stevedlawrence!

ktoso added a commit that referenced this pull request Jun 22, 2015
Convert Scala @throws annotations to Java throws clauses
@ktoso ktoso merged commit 256266f into lightbend:master Jun 22, 2015
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.

3 participants