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] - doc: Change old Lean 3 commands to Lean 4 in implementation notes #10707

Closed
wants to merge 2 commits into from

Conversation

OmarMohsenGit
Copy link
Collaborator

@OmarMohsenGit OmarMohsenGit commented Feb 18, 2024

I changed Lean's 3 old "variables" command to Lean's 4 command "variable" in some implementation notes.
I might have missed some

@OmarMohsenGit OmarMohsenGit changed the title Change some old lean 3 commands in implementation notes doc: Change old Lean 3 commands to Lean 4 in implementation notes Feb 18, 2024
@adomani adomani added the awaiting-author A reviewer has asked the author a question or requested changes label Feb 19, 2024
@riccardobrasca
Copy link
Member

Taking into account Damiano's comment LGTM, thanks!

bors d+

@mathlib-bors
Copy link

mathlib-bors bot commented Feb 19, 2024

✌️ OmarMohsenGit can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

Co-authored-by: damiano <adomani@gmail.com>
@ADedecker ADedecker removed the awaiting-author A reviewer has asked the author a question or requested changes label Feb 19, 2024
@OmarMohsenGit
Copy link
Collaborator Author

bors r+

mathlib-bors bot pushed a commit that referenced this pull request Feb 20, 2024
…0707)

I changed Lean's 3 old "variables" command to Lean's 4 command "variable" in some implementation notes.
I might have missed some

Co-authored-by: Omar Mohsen <36500353+OmarMohsenGit@users.noreply.github.com>
@mathlib-bors
Copy link

mathlib-bors bot commented Feb 20, 2024

Build failed:

@riccardobrasca
Copy link
Member

It seems like a random failure, let's try again.

bors merge

@github-actions github-actions bot added the ready-to-merge This PR has been sent to bors. label Feb 20, 2024
mathlib-bors bot pushed a commit that referenced this pull request Feb 20, 2024
…0707)

I changed Lean's 3 old "variables" command to Lean's 4 command "variable" in some implementation notes.
I might have missed some

Co-authored-by: Omar Mohsen <36500353+OmarMohsenGit@users.noreply.github.com>
@mathlib-bors
Copy link

mathlib-bors bot commented Feb 20, 2024

Pull request successfully merged into master.

Build succeeded:

@mathlib-bors mathlib-bors bot changed the title doc: Change old Lean 3 commands to Lean 4 in implementation notes [Merged by Bors] - doc: Change old Lean 3 commands to Lean 4 in implementation notes Feb 20, 2024
@mathlib-bors mathlib-bors bot closed this Feb 20, 2024
@mathlib-bors mathlib-bors bot deleted the my_new_branch branch February 20, 2024 11:30
thorimur pushed a commit that referenced this pull request Feb 24, 2024
…0707)

I changed Lean's 3 old "variables" command to Lean's 4 command "variable" in some implementation notes.
I might have missed some

Co-authored-by: Omar Mohsen <36500353+OmarMohsenGit@users.noreply.github.com>
thorimur pushed a commit that referenced this pull request Feb 26, 2024
…0707)

I changed Lean's 3 old "variables" command to Lean's 4 command "variable" in some implementation notes.
I might have missed some

Co-authored-by: Omar Mohsen <36500353+OmarMohsenGit@users.noreply.github.com>
riccardobrasca pushed a commit that referenced this pull request Mar 1, 2024
…0707)

I changed Lean's 3 old "variables" command to Lean's 4 command "variable" in some implementation notes.
I might have missed some

Co-authored-by: Omar Mohsen <36500353+OmarMohsenGit@users.noreply.github.com>
dagurtomas pushed a commit that referenced this pull request Mar 22, 2024
…0707)

I changed Lean's 3 old "variables" command to Lean's 4 command "variable" in some implementation notes.
I might have missed some

Co-authored-by: Omar Mohsen <36500353+OmarMohsenGit@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
delegated 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

4 participants