Skip to content

[Merged by Bors] - feat: extend pp.links to support Pi, Prop, Type, and Sort#778

Closed
eric-wieser wants to merge 3 commits intomasterfrom
eric-wieser/builtin-links
Closed

[Merged by Bors] - feat: extend pp.links to support Pi, Prop, Type, and Sort#778
eric-wieser wants to merge 3 commits intomasterfrom
eric-wieser/builtin-links

Conversation

@eric-wieser
Copy link
Copy Markdown
Member

@eric-wieser eric-wieser commented Nov 4, 2022

This uses U+E003 (the next available private use character after the ones claimed in #89) to prefix the names:

  • pi
  • forall
  • function
  • implies
  • Sort
  • Prop
  • Type

The motivation is to be able to link these ideas in doc-gen.

This uses `U+E003` (the next available private use character) to prefix the names:

* `pi`
* `forall`
* `function`
* `implies`
* `Sort`
* `Prop`
* `Type`

The motivation is to be able to link these ideas in doc-gen.
@eric-wieser eric-wieser requested a review from gebner November 4, 2022 00:37
@eric-wieser eric-wieser force-pushed the eric-wieser/builtin-links branch from 55a0032 to ab29f49 Compare November 4, 2022 00:53
@gebner
Copy link
Copy Markdown
Member

gebner commented Nov 11, 2022

bors r+

bors bot pushed a commit that referenced this pull request Nov 11, 2022
This uses `U+E003` (the next available private use character after the ones claimed in #89) to prefix the names:

* `pi`
* `forall`
* `function`
* `implies`
* `Sort`
* `Prop`
* `Type`

The motivation is to be able to link these ideas in doc-gen.
@gebner
Copy link
Copy Markdown
Member

gebner commented Nov 11, 2022

Thanks!

@bors
Copy link
Copy Markdown

bors bot commented Nov 11, 2022

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: extend pp.links to support Pi, Prop, Type, and Sort [Merged by Bors] - feat: extend pp.links to support Pi, Prop, Type, and Sort Nov 11, 2022
@bors bors bot closed this Nov 11, 2022
@bors bors bot deleted the eric-wieser/builtin-links branch November 11, 2022 18:50
@eric-wieser
Copy link
Copy Markdown
Member Author

Mind looking at the related leanprover-community/doc-gen#173 too?

@gebner
Copy link
Copy Markdown
Member

gebner commented Nov 11, 2022

The doc-gen PR is also a 👍 from me, but we'll first need to bump mathlib to 3.49. leanprover-community/mathlib3#17470

@eric-wieser
Copy link
Copy Markdown
Member Author

The doc-gen PR doesn't need a lean bump

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.

2 participants