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 Algebra.Star.Basic #1331

Closed
wants to merge 42 commits into from

Conversation

qawbecrdtey
Copy link
Collaborator

@qawbecrdtey qawbecrdtey commented Jan 4, 2023

No description provided.

@qawbecrdtey qawbecrdtey added WIP Work in progress mathlib-port This is a port of a theory file from mathlib. labels Jan 4, 2023
@qawbecrdtey qawbecrdtey added the help-wanted The author needs attention to resolve issues label Jan 4, 2023
@dupuisf dupuisf changed the title Port/algebra.star.basic feat: port Algebra.Star.Basic Jan 4, 2023
@semorrison semorrison added the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Jan 5, 2023
@qawbecrdtey qawbecrdtey removed the merge-conflict The PR has a merge conflict with master, and needs manual merging. label Jan 5, 2023
qawbecrdtey and others added 2 commits January 7, 2023 19:23
@qawbecrdtey qawbecrdtey added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jan 7, 2023
--#align star_rat_cast star_rat_cast
section
-- Porting note: This takes too long
set_option maxHeartbeats 0
Copy link
Member

Choose a reason for hiding this comment

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

I have no clue why it needs so much time. But we should investigate this, because the proofs look pretty innocent.

qawbecrdtey and others added 2 commits January 8, 2023 02:44
Co-authored-by: Chris Hughes <33847686+ChrisHughes24@users.noreply.github.com>
Co-authored-by: Chris Hughes <33847686+ChrisHughes24@users.noreply.github.com>
Mathlib/Algebra/Star/Basic.lean Outdated Show resolved Hide resolved
Mathlib/Algebra/Star/Basic.lean Outdated Show resolved Hide resolved
@dupuisf dupuisf added awaiting-author A reviewer has asked the author a question or requested changes and removed awaiting-review The author would like community review of the PR labels Jan 8, 2023
qawbecrdtey and others added 3 commits January 8, 2023 13:25
Co-authored-by: Ruben Van de Velde <65514131+Ruben-VandeVelde@users.noreply.github.com>
@qawbecrdtey qawbecrdtey added awaiting-review The author would like community review of the PR and removed awaiting-author A reviewer has asked the author a question or requested changes labels Jan 8, 2023
@qawbecrdtey qawbecrdtey dismissed Ruben-VandeVelde’s stale review January 8, 2023 09:06

These changs have been applied

@ChrisHughes24
Copy link
Member

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 Jan 9, 2023
bors bot pushed a commit that referenced this pull request Jan 9, 2023
Co-authored-by: Frédéric Dupuis <dupuisf@iro.umontreal.ca>
Co-authored-by: qawbecrdtey <40463813+qawbecrdtey@users.noreply.github.com>
Co-authored-by: Siddhartha Gadgil <siddhartha.gadgil@gmail.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
@bors
Copy link

bors bot commented Jan 9, 2023

Pull request successfully merged into master.

Build succeeded:

@bors bors bot changed the title feat: port Algebra.Star.Basic [Merged by Bors] - feat: port Algebra.Star.Basic Jan 9, 2023
@bors bors bot closed this Jan 9, 2023
@bors bors bot deleted the port/Algebra.Star.Basic branch January 9, 2023 09:45
jcommelin pushed a commit that referenced this pull request Jan 23, 2023
Co-authored-by: Frédéric Dupuis <dupuisf@iro.umontreal.ca>
Co-authored-by: qawbecrdtey <40463813+qawbecrdtey@users.noreply.github.com>
Co-authored-by: Siddhartha Gadgil <siddhartha.gadgil@gmail.com>
Co-authored-by: Scott Morrison <scott.morrison@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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

7 participants