-
Notifications
You must be signed in to change notification settings - Fork 27
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
False positive #89
Comments
Can you share you weeder Dhall config, along with sample weeds and why you think they are false positives? |
First, thanks for helping me. I used several different dhall file without any different behavior, last/current one is:
It really list every functions of the whole program. As the program is somehow small (just a couple of files), it is easy for me to verify that weeder is wrong. For example, Here is also some extra informations. What is weird is the output of weeder: src/Business.hs:82: logUserMonthSimulation
src/Business.hs:63: processMonth
src/Business.hs:29: processingBalances
src/Business.hs:56: processingConduit
src/Business.hs:95: updateUserForMonth and it stops there, without any detail or number of weeds but the program exit with a 1 ( If I compare with another project (different version of everything, weeder, stack, ghc) (...)
src/Commons/Model/User/Permission.hs:16:1: error: allWritePermissions is unused
14 ┃
15 ┃ allWritePermissions :: [PowerUser] -> [Permission]
16 ┃ allWritePermissions = fmap Write
Delete this definition or add ‘Commons.Model.User.Permission.allWritePermissions’ as a root to fix this error.
Weeds detected: 8 If there is not dead code on the second project obvioulsy the output of weeder is NB: note that for some reason I had to position The difference of weeder behavior on the first project is very weird and as I do not get the number of weeds maybe it could tell us that it crash? |
I'm also getting some definite false positives with ghc 8.10.7 and weeder 2.1.3 , but in a proprietary codebase. |
Any news over here? |
Sorry, I haven't had a chance to look. |
I am facing an issue with weeder, it literally says that all of my functions are unused.
Version used:
It was working fine a couple of weeks ago and I just discover this weird behavior during:
The hie files are well generated, HLS is able to use them correctly in my editor (so they should be fine).
I end up trying differents configs in the dhall without any success, from my old configuration to the default one propose in this repo.
By chance do you have any tips or hint about this weird situation?
The text was updated successfully, but these errors were encountered: