Skip to content

Problems with change of Semilattice operation name #2166

Closed
@MatthewDaggitt

Description

@MatthewDaggitt

First issue with RC 1. Changelog doesn't mention that the name of the operation in Semilattice changed from _∧_ to _∙_.

More problematically the names in IsSemilattice have not been changed 😢

open IsBand isBand public

Activity

added this to the v2.0 milestone on Oct 19, 2023
changed the title [-]Missing CHANGELOG entry for change of `Semilattice` operation name[/-] [+]Problems with change of `Semilattice` operation name[/+] on Oct 19, 2023
Taneb

Taneb commented on Oct 26, 2023

@Taneb
Member

I'd like to address this more aggressively, by tying it with #2138 and removing the undirected IsLattice etc structures entirely.

jamesmckinna

jamesmckinna commented on Oct 26, 2023

@jamesmckinna
Contributor

Don't remove: deprecate!

jamesmckinna

jamesmckinna commented on Nov 11, 2023

@jamesmckinna
Contributor

@Taneb Any progress on this issue/PR?

jamesmckinna

jamesmckinna commented on Nov 20, 2023

@jamesmckinna
Contributor

Cf. #2178 what is the operator symbol supposed to be in Semilattice?
I'm starting to wonder/panic whether that PR actually did the right thing?

MatthewDaggitt

MatthewDaggitt commented on Nov 24, 2023

@MatthewDaggitt
ContributorAuthor

Should be the dot symbol. In the interests of getting the next RC out the door I'm taking this one over.

Taneb

Taneb commented on Nov 24, 2023

@Taneb
Member

Sorry, I've been sick lately. I'm only just getting enough brainpower back to write Agda now

MatthewDaggitt

MatthewDaggitt commented on Nov 25, 2023

@MatthewDaggitt
ContributorAuthor

No worries, I hope you have a swift recovery!

jamesmckinna

jamesmckinna commented on Nov 25, 2023

@jamesmckinna
Contributor

Get well soon! I've approved Matthew's PR in the interim.

added a commit that references this issue on Nov 26, 2023

Fixes #2166 by fixing names in `IsSemilattice` (#2211)

b9e132e
added a commit that references this issue on Jul 10, 2024

Fixes #2166 by fixing names in `IsSemilattice` (#2211)

56ed8fe
added a commit that references this issue on Sep 7, 2024
4144346
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

No type

Projects

No projects

Relationships

None yet

    Development

    Participants

    @Taneb@MatthewDaggitt@jamesmckinna

    Issue actions

      Problems with change of `Semilattice` operation name · Issue #2166 · agda/agda-stdlib