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

deprecate do: meaning do (): + misc cleanup #20927

Merged
merged 7 commits into from
Dec 6, 2022

Conversation

metagn
Copy link
Collaborator

@metagn metagn commented Nov 26, 2022

@Varriount Varriount requested a review from Araq November 26, 2022 16:28
@Varriount Varriount added the Requires Araq To Merge PR should only be merged by Araq label Nov 26, 2022
@metagn
Copy link
Collaborator Author

metagn commented Nov 29, 2022

CI is fine now

@Araq Araq merged commit 6d8cf25 into nim-lang:devel Dec 6, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Dec 6, 2022

Thanks for your hard work on this PR!
The lines below are statistics of the Nim compiler built from 6d8cf25

Hint: mm: orc; opt: speed; options: -d:release
165480 lines; 8.500s; 612.676MiB peakmem

survivorm pushed a commit to survivorm/Nim that referenced this pull request Feb 28, 2023
* test disable do: block lambda lifting

* fix last test [skip ci]

* deprecate `do:` meaning `do ():` + misc cleanup

closes nim-lang/RFCs#486

* oops

* fix

* no idea what could be causing nimsuggest failure other than this

* ensure ci works
capocasa pushed a commit to capocasa/Nim that referenced this pull request Mar 31, 2023
* test disable do: block lambda lifting

* fix last test [skip ci]

* deprecate `do:` meaning `do ():` + misc cleanup

closes nim-lang/RFCs#486

* oops

* fix

* no idea what could be causing nimsuggest failure other than this

* ensure ci works
bung87 pushed a commit to bung87/Nim that referenced this pull request Jul 29, 2023
* test disable do: block lambda lifting

* fix last test [skip ci]

* deprecate `do:` meaning `do ():` + misc cleanup

closes nim-lang/RFCs#486

* oops

* fix

* no idea what could be causing nimsuggest failure other than this

* ensure ci works
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Requires Araq To Merge PR should only be merged by Araq
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Style check/remove do: blocks lifted to lambdas
3 participants