Skip to content

ci: run missinglinkCheck in CI #199

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 7 commits into from
Mar 8, 2021
Merged

ci: run missinglinkCheck in CI #199

merged 7 commits into from
Mar 8, 2021

Conversation

sideeffffect
Copy link
Contributor

No description provided.

@sideeffffect
Copy link
Contributor Author

sideeffffect commented Apr 1, 2020

java.lang.OutOfMemoryError: GC overhead limit exceeded

🤦‍♂️ what a great late April Fools joke
oh well, will sort out tomorrow

@jakubjanecek
Copy link
Collaborator

I wanted to enable this too but it was quite immature so I waited. We can test it now if it works well...

@sideeffffect
Copy link
Contributor Author

btw I finally decided to bite the bullet and start using missinglink, because there's been a new release recently, which I hoped would improve it's practical use (I think it introduced the suppression for FPs -- without the suppression it's not usable at all) and had been bitten by Scala TypeLevel lib binary incompatibility (again 🤬)

also, sbt-missinglink is developed by Scala Center (like Scalafix)

@jakubjanecek
Copy link
Collaborator

I think it's time to merge this ;)

@mergify mergify bot merged commit fbf36c0 into master Mar 8, 2021
@mergify mergify bot deleted the sideeffffect-patch-1 branch March 8, 2021 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants