Skip to content

[ fix ] Relation.Nullary.Decidable.Core names for combinators#2843

Merged
jamesmckinna merged 7 commits intoagda:masterfrom
jamesmckinna:decidable-names
Oct 28, 2025
Merged

[ fix ] Relation.Nullary.Decidable.Core names for combinators#2843
jamesmckinna merged 7 commits intoagda:masterfrom
jamesmckinna:decidable-names

Conversation

@jamesmckinna
Copy link
Collaborator

Fixes #2842

Issues:

  • lots of knock-on viscosity
  • lots of other occurrences of -dec as a suffix which could/should be changed to ?

Copy link
Member

@gallais gallais left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for that. Wondering whether there are a few more opportunities for consistent naming.

@jamesmckinna
Copy link
Collaborator Author

jamesmckinna commented Oct 26, 2025

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 -dec"... but I thought it better to handle those downstream.

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.

But happy to do so if reviewers agree, although Suggest we move such things downstream, as I think/worry it might open many more cans of worms? The ones you identify arise as direct consequences of this PR, but I fear the general pattern (and its violation) may be even more widespread...

@JacquesCarette
Copy link
Collaborator

Wow, I was surprised that you even attempted this as v2.4.

Copy link
Collaborator

@JacquesCarette JacquesCarette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@jamesmckinna
Copy link
Collaborator Author

jamesmckinna commented Oct 28, 2025

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.

@jamesmckinna jamesmckinna added this pull request to the merge queue Oct 28, 2025
Merged via the queue into agda:master with commit dd64a3a Oct 28, 2025
18 of 22 checks passed
@jamesmckinna
Copy link
Collaborator Author

Ugh. Forgot to to fix the alignment in CHANGELOG.

@MatthewDaggitt
Copy link
Collaborator

That's okay, there's always a tidying up pass before release.

plt-amy pushed a commit that referenced this pull request Feb 9, 2026
* refactor: fixes #2842

* refactor: knock-ons

* refactor: more knock-ons

* refactor: resolve merge conflict following #2793
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use consistent names for Decidable combinators

4 participants