[ fix ] Relation.Nullary.Decidable.Core names for combinators#2843
[ fix ] Relation.Nullary.Decidable.Core names for combinators#2843jamesmckinna merged 7 commits intoagda:masterfrom
Relation.Nullary.Decidable.Core names for combinators#2843Conversation
gallais
left a comment
There was a problem hiding this comment.
Thank you for that. Wondering whether there are a few more opportunities for consistent naming.
yes, hence the OP reference to "lots of other occurrences of I already encountered the need for some careful qualified importing in order to avoid name clashes, and I didn't want to get into deeper rabbit holes by widening the impact of this PR.
|
|
Wow, I was surprised that you even attempted this as v2.4. |
JacquesCarette
left a comment
There was a problem hiding this comment.
While I agree that there are quite a few more things that ought to be changed, I think that it is fine to do this kind of work incrementally. So what @gallais points out could form the basis of 2+ future PRs. Similarly with the wart that I point out.
See also:
UPDATED this PR closes #2842 so lifted out the additional comments/examples to #2846. Merging now. |
|
Ugh. Forgot to to fix the alignment in |
|
That's okay, there's always a tidying up pass before release. |
Fixes #2842
Issues:
-decas a suffix which could/should be changed to?