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

Add scala-clippy advice #54

Open
sellout opened this issue Dec 19, 2016 · 2 comments
Open

Add scala-clippy advice #54

sellout opened this issue Dec 19, 2016 · 2 comments

Comments

@sellout
Copy link
Contributor

sellout commented Dec 19, 2016

Matryoshka could definitely benefit from library-specific advice when it comes to compiler errors. See here for how: https://github.com/softwaremill/scala-clippy#library-specific-advice

@ShaneDelmore
Copy link

I'll take this one. Been looking for a library to contribute advice to since I added the feature to clippy. If anyone has common gotchas that could benefit from advice sent in response to a compiler error add them here and I will add the advice and tests. Missing imports are a common area that can benefit.

@sellout
Copy link
Contributor Author

sellout commented Jan 11, 2017

@smungee received the following error: “matryoshka.Recursive.Ops[matryoshka.data.Fix[$file.ExprTest.Expr],$file.ExprTest.Expr]#transAna.type does not take parameters” and the explanation I’d like to see is something like “transAna requires a type parameter specifying the result type. In this case, it may be transAna[Fix[Expr]],” (where both “transAna” and “Fix[Expr]” are extracted from the error message (if possible)).

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

2 participants