Cabal-syntax: trade TypeFamilies for FunctionalDependencies #11355
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The overarching goal is to make
Cabal-syntaxcompilable by MicroHs, which does not supportTypeFamilies, but does supportFunctionalDependencies. See #11341 for more context.The only type family Cabal uses is
This PR changes it to
Ignoring my original motivation for a moment, I think type families do not really bear their weight for
class IsNode. Fundeps are more explicit and better to work with: I don't need to evaluatetype Keyin my head to figure out what's exactly the key, it's right there in type signatures now.I'll update changelog once this passes CI.
Template Α: This PR modifies behaviour or interface
Include the following checklist in your PR:
significance: significantin the changelog file.