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] - feat: Port Logic.Nonempty #487

Closed
wants to merge 21 commits into from

Conversation

pechersky
Copy link
Collaborator

@pechersky pechersky commented Oct 21, 2022

  • Deleted nonempty_empty since std4 now has that as
    not_nonempty_empty and as a simp lemma
  • Added an import to access Zero for Zero.Nonempty
  • Could not provide a One.Nonempty because One isn't defined yet
  • Used mathport's implicit type params, not sure what the style guidelines are
  • Gave previously anonymous instances names: Prod.Nonempty, Pi.Nonempty

semorrison and others added 13 commits October 20, 2022 15:42
[] depends on: #484

- Deleted `nonempty_empty` since `std4` now has that as
`not_nonempty_empty` and as a simp lemma
- Added an import to access `Zero` for `Zero.Nonempty`
- Could not provide a `One.Nonempty` because `One` isn't defined yet
- Used mathport's implicit type params, not sure what the style guidelines are
- Gave previously anonymous instances names: `Prod.Nonempty`, `Pi.Nonempty`
@pechersky pechersky changed the title Pechersky/port-logic-nonempty Port Logic.Nonempty Oct 21, 2022
@digama0 digama0 changed the title Port Logic.Nonempty feat: Port Logic.Nonempty Oct 23, 2022
@semorrison semorrison added the blocked-by-other-PR This PR depends on another PR which is still in the queue. label Oct 23, 2022
@pechersky pechersky added awaiting-review The author would like community review of the PR and removed blocked-by-other-PR This PR depends on another PR which is still in the queue. labels Oct 24, 2022
@semorrison
Copy link
Contributor

bors merge

bors bot pushed a commit that referenced this pull request Oct 25, 2022
- [x] depends on: #484


* Deleted `nonempty_empty` since `std4` now has that as
`not_nonempty_empty` and as a simp lemma
* Added an import to access `Zero` for `Zero.Nonempty`
* Could not provide a `One.Nonempty` because `One` isn't defined yet
* Used mathport's implicit type params, not sure what the style guidelines are
* Gave previously anonymous instances names: `Prod.Nonempty`, `Pi.Nonempty`

Co-authored-by: Jakob von Raumer <jakob@von-raumer.de>
Co-authored-by: Yakov Pechersky <ypechersky@treeline.bio>
@bors
Copy link

bors bot commented Oct 25, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: Port Logic.Nonempty [Merged by Bors] - feat: Port Logic.Nonempty Oct 25, 2022
@bors bors bot closed this Oct 25, 2022
@bors bors bot deleted the pechersky/port-logic-nonempty branch October 25, 2022 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review The author would like community review of the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants