Skip to content

fix: Remove WartRemover as it was not working properly on Scala 2.13 #147

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

Merged
merged 1 commit into from
Feb 11, 2020

Conversation

jakubjanecek
Copy link
Collaborator

build: Update dependencies

refactor: Small improvements

build: Update dependencies

refactor: Small improvements
@jakubjanecek jakubjanecek added bug Something isn't working dependency-update Update of dependencies (Scala Steward) build Changes to the build system labels Feb 11, 2020
@jakubjanecek jakubjanecek self-assigned this Feb 11, 2020
@jakubjanecek
Copy link
Collaborator Author

After switching to Scala 2.13 the release failed: https://github.com/avast/scala-server-toolkit/runs/436430636?check_suite_focus=true#step:8:763

I spent a lot of time debugging the problems (trying to tweak different SBT settings), trying to find some existing problem in SBT or Scaladoc or something like that related to Scala 2.13 but I was unsuccessful. Finally I found out that the problem lies in WartRemover which even though supports Scala 2.13 it did not work well together with SemanticDB which is necessary for Scalafix. I consider Scalafix better for the future so I have decided to remove WartRemover altogether.

@jakubjanecek jakubjanecek merged commit ade8a83 into master Feb 11, 2020
@jakubjanecek jakubjanecek deleted the fix/ReplaceWartRemover branch February 11, 2020 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working build Changes to the build system dependency-update Update of dependencies (Scala Steward)
Development

Successfully merging this pull request may close these issues.

2 participants