Remove hashable dependency
See #10140
- [x] Patches conform to the coding conventions.
- [ ] Is this a PR that fixes CI? If so, it will need to be backported to older cabal release branches (ask maintainers for directions).
Removing hashable from the test suite nerd-sniped me into writing a bit of whacky arbitrary machinery, but I think it's not necessarily worse than abusing hashable for this.
I added some minimal comments to the code as well.
I'd hope for a smaller diff if we just had some of the compat code added to some sort of Prelude or Compat module
IMO, Distribution.Client.HashValue is this compat module. It just wasn't used in FileMonitor yet...
IMO, Distribution.Client.HashValue is this compat module. It just wasn't used in FileMonitor yet...
Oh, that's good, thank you!
This needs a manual rebase on top of master, sorry @jaspervdj. Do you have time for this or should we try to take care of it?
Actually, this was trivial, don't worry. Should merge soon.
@mergify backport 3.14
backport 3.14
✅ Backports have been created
- #10536 Backport #10316: Remove hashable dependency has been created for branch
3.14but encountered conflicts