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

Improved array! macro diagnostics. #3922

Merged
merged 1 commit into from
Aug 22, 2023
Merged

Conversation

orizi
Copy link
Collaborator

@orizi orizi commented Aug 16, 2023

@orizi orizi requested a review from spapinistarkware August 16, 2023 13:59
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/e634c26d branch from 04e92f0 to 457fa5e Compare August 16, 2023 15:49
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from 010d6f1 to 0e4c8b7 Compare August 16, 2023 15:49
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/e634c26d branch from 457fa5e to e05be3e Compare August 16, 2023 17:29
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from 0e4c8b7 to 673b843 Compare August 16, 2023 17:29
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/e634c26d branch from e05be3e to fc7942b Compare August 17, 2023 06:44
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from 673b843 to aa7c931 Compare August 17, 2023 06:44
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch 2 times, most recently from 63aa6f9 to f72c4ce Compare August 17, 2023 13:41
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/e634c26d branch from 886549d to 4df1084 Compare August 17, 2023 14:05
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch 2 times, most recently from 3cd1420 to b349bd1 Compare August 17, 2023 17:35
@orizi orizi changed the base branch from pr/orizi/orizi/inline-macro/e634c26d to main August 20, 2023 13:29
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from b349bd1 to 86f9d3e Compare August 20, 2023 13:29
@orizi orizi changed the title Improved array! macro diagnostics. Include missing patch name in panic. Aug 21, 2023
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from 86f9d3e to 4045c7d Compare August 21, 2023 08:27
@orizi orizi changed the title Include missing patch name in panic. Improved array! macro diagnostics. Aug 21, 2023
@orizi orizi changed the base branch from main to pr/orizi/orizi/inline-macro/e634c26d August 21, 2023 08:28
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from 4045c7d to 82b2a63 Compare August 22, 2023 07:01
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/e634c26d branch from 2c380c7 to 39749a3 Compare August 22, 2023 07:52
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch 4 times, most recently from faf3db6 to 9f35ca1 Compare August 22, 2023 11:38
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/e634c26d branch from 6023663 to 67b3c26 Compare August 22, 2023 11:38
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from 9f35ca1 to bb1596b Compare August 22, 2023 12:03
@orizi orizi changed the base branch from pr/orizi/orizi/inline-macro/e634c26d to main August 22, 2023 12:52
@orizi orizi force-pushed the pr/orizi/orizi/inline-macro/944d2078 branch from bb1596b to 89ac186 Compare August 22, 2023 12:52
Copy link
Contributor

@gilbens-starkware gilbens-starkware left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @spapinistarkware)

@orizi orizi added this pull request to the merge queue Aug 22, 2023
Merged via the queue into main with commit 9ce168c Aug 22, 2023
@orizi orizi deleted the pr/orizi/orizi/inline-macro/944d2078 branch August 22, 2023 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants