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] - Add missing readmes #6031

Closed
wants to merge 4 commits into from
Closed

Conversation

eric-wieser
Copy link
Member

These are copied with minimal modification from mathlib3.


Open in Gitpod

@eric-wieser eric-wieser added documentation Improvements or additions to documentation awaiting-review mathlib-port This is a port of a theory file from mathlib. labels Jul 20, 2023
Archive/Imo/README.md Outdated Show resolved Hide resolved
Archive/Imo/README.md Outdated Show resolved Hide resolved
Archive/README.md Outdated Show resolved Hide resolved
Try to adhere to the guidelines for mathlib. They will be much less strictly enforced for the archive, but we still want you to adhere to all the conventions that make maintenance easier. This ensures that when mathlib is changing, the mathlib maintainers can fix these contributions without much effort. Here are the guidelines:
- The [style guide](https://leanprover-community.github.io/contribute/style.html) for contributors.
- The [git commit conventions](https://github.com/leanprover-community/lean4/blob/master/doc/commit_convention.md).
- You don't have to follow the [naming conventions](https://leanprover-community.github.io/contribute/naming.html).
Copy link
Member

Choose a reason for hiding this comment

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

Still about Lean 3.

Copy link
Member Author

@eric-wieser eric-wieser Jul 20, 2023

Choose a reason for hiding this comment

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

This link is supposed to lead to the Lean4 one though, right? Which is to say, the problem is at the page it links to.

Co-authored-by: Yury G. Kudryashov <urkud@urkud.name>
Archive/Imo/README.md Outdated Show resolved Hide resolved
Archive/Imo/README.md Outdated Show resolved Hide resolved
Co-authored-by: Junyan Xu <junyanxu.math@gmail.com>
Copy link
Member

@jcommelin jcommelin left a comment

Choose a reason for hiding this comment

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

Thanks 🎉

bors merge

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added ready-to-merge This PR has been sent to bors. and removed awaiting-review labels Jul 21, 2023
bors bot pushed a commit that referenced this pull request Jul 21, 2023
These are copied with minimal modification from mathlib3.
@bors
Copy link

bors bot commented Jul 21, 2023

Build failed (retrying...):

bors bot pushed a commit that referenced this pull request Jul 21, 2023
These are copied with minimal modification from mathlib3.
@bors
Copy link

bors bot commented Jul 21, 2023

Pull request successfully merged into master.

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot changed the title Add missing readmes [Merged by Bors] - Add missing readmes Jul 21, 2023
@bors bors bot closed this Jul 21, 2023
@bors bors bot deleted the add-missing-readmes branch July 21, 2023 16:03
fgdorais pushed a commit that referenced this pull request Jul 25, 2023
These are copied with minimal modification from mathlib3.
semorrison pushed a commit that referenced this pull request Aug 14, 2023
These are copied with minimal modification from mathlib3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation mathlib-port This is a port of a theory file from mathlib. 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

5 participants