Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Merged by Bors] - refactor(Control/ULiftable): use outParam and adjust universe orders #8849

Closed
wants to merge 2 commits into from

Conversation

eric-wieser
Copy link
Member

The most useful universe argument to up and down (the one in the ULift type) is now first.

Combined with the outParam, this makes using ULiftable.up more ergonomic downstream, removing a handful of type annotations.


Open in Gitpod

@eric-wieser eric-wieser added awaiting-review The author would like community review of the PR awaiting-CI labels Dec 6, 2023
@semorrison
Copy link
Contributor

bors merge

@github-actions github-actions bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review The author would like community review of the PR labels Dec 12, 2023
mathlib-bors bot pushed a commit that referenced this pull request Dec 12, 2023
#8849)

The most useful universe argument to `up` and `down` (the one in the `ULift` type) is now first.

Combined with the `outParam`, this makes using `ULiftable.up` more ergonomic downstream, removing a handful of type annotations.
@mathlib-bors
Copy link

mathlib-bors bot commented Dec 12, 2023

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title refactor(Control/ULiftable): use outParam and adjust universe orders [Merged by Bors] - refactor(Control/ULiftable): use outParam and adjust universe orders Dec 12, 2023
@mathlib-bors mathlib-bors bot closed this Dec 12, 2023
@mathlib-bors mathlib-bors bot deleted the eric-wieser/uliftable-up-outparam branch December 12, 2023 01:34
awueth pushed a commit that referenced this pull request Dec 19, 2023
#8849)

The most useful universe argument to `up` and `down` (the one in the `ULift` type) is now first.

Combined with the `outParam`, this makes using `ULiftable.up` more ergonomic downstream, removing a handful of type annotations.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has been sent to bors.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants