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

Drop reliance on quill-util and make another upstream library to encapsulate ScalaFMT needs #19

Open
deusaquilus opened this issue Dec 16, 2022 · 0 comments

Comments

@deusaquilus
Copy link
Contributor

Want to be able to optionally use scalafmt to format code if it is on the path. This is indispensable for .info calls etc... but want to make this optional. Also, don't want to directly compile anything that has to do with scalafmt directly because it has a nasty tendency to spew hundreds of irrelevant protobuff errors. The ideal way to do this would be to create an upstream library that does what Quill's ScalafmtFormat class does. Essentially have a generic FmtUtils library in the ZIO namespace that does basically what quill-util does.

I added a branch to how how Scalafmt could be pulled in directly (https://github.com/zio/zio-direct/tree/pull-in-scalafmt-directly) and it successfully copiles but if anything is wrong with the compile dozens of protobuff errors will spew to the console. Therefore I do not want to use that branch.

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

No branches or pull requests

1 participant