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

fix warning in TransactionTest #2762

Merged
merged 1 commit into from
Jul 11, 2023
Merged

fix warning in TransactionTest #2762

merged 1 commit into from
Jul 11, 2023

Conversation

xuwei-k
Copy link
Contributor

@xuwei-k xuwei-k commented Jul 11, 2023

avoid local class

[warn] -- Unchecked Warning: /home/runner/work/slick/slick/slick-testkit/src/main/scala/com/typesafe/slick/testkit/tests/TransactionTest.scala:27:55 
[warn] 27 |      } yield ()).transactionally.failed.map(_ should (_.isInstanceOf[ExpectedException]))
[warn]    |                                                       ^
[warn]    |the type test for ExpectedException cannot be checked at runtime because it's a local class
[warn] -- Unchecked Warning: /home/runner/work/slick/slick/slick-testkit/src/main/scala/com/typesafe/slick/testkit/tests/TransactionTest.scala:53:71 
[warn] 53 |      } yield ()).transactionally.transactionally.failed.map(_ should (_.isInstanceOf[ExpectedException]))
[warn]    |                                                                       ^
[warn]    |the type test for ExpectedException cannot be checked at runtime because it's a local class

@github-actions
Copy link
Contributor

There are no incompatible changes

@nafg nafg merged commit c4467c2 into slick:main Jul 11, 2023
@nafg
Copy link
Member

nafg commented Jul 11, 2023

Thanks!

@xuwei-k xuwei-k deleted the patch-1 branch July 11, 2023 00:57
@nafg nafg added this to the 3.5.0 milestone Nov 20, 2023
@nafg nafg added build Change to the build definition and dependencies refactor labels Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Change to the build definition and dependencies refactor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants