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

Reduce compilation warnings in tests #4083

Merged
merged 9 commits into from
Apr 2, 2022

Conversation

satorg
Copy link
Contributor

@satorg satorg commented Dec 11, 2021

This is an attempt to improve code hygiene a bit by reducing amount of warnings while compiling the testsJVM, testsJS and testsNative modules.

No tests are removed nor disabled, no other changes besides the tests* modules. But some tests were tossed over different suites or suite mixins.

Caveat: because @nowarn annotation is not supported in Scala 3.0.x, for this cross-build configuration most of the warnings will still remain. But things should get improved once the project upgrades to Scala 3.1.x.

Note: also in all changed files the "Organize Imports" (vs-code+metals) command was applied.

@satorg satorg force-pushed the reduce-tests-warnings branch from b09ec19 to 98478c8 Compare December 16, 2021 19:51
@satorg
Copy link
Contributor Author

satorg commented Dec 17, 2021

This PR is still awaiting its hero who would dare to review it :)

@satorg satorg force-pushed the reduce-tests-warnings branch from 98478c8 to 7e21c4a Compare December 25, 2021 20:38
@satorg satorg force-pushed the reduce-tests-warnings branch from 7e21c4a to 045ac91 Compare April 2, 2022 00:10
@satorg
Copy link
Contributor Author

satorg commented Apr 2, 2022

Hi folks! May I draw a fraction of your attention to this PR please? There's nothing dangerous with it – it just reduces compilation warnings from tests which can help to alleviate developer experience tremendously while they're working on any tests as well as coding in any IDE.

Copy link
Member

@rossabaker rossabaker left a comment

Choose a reason for hiding this comment

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

Thanks!

@johnynek johnynek merged commit 8200dbf into typelevel:main Apr 2, 2022
@satorg satorg deleted the reduce-tests-warnings branch April 2, 2022 01:28
@satorg
Copy link
Contributor Author

satorg commented Apr 2, 2022

@rossabaker @johnynek thank you guys!

@johnynek
Copy link
Contributor

johnynek commented Apr 2, 2022

Thank you! Sorry for the latency!

@satorg satorg mentioned this pull request Apr 4, 2022
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.

4 participants