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

update nimble #22814

Merged
merged 1 commit into from
Oct 11, 2023
Merged

update nimble #22814

merged 1 commit into from
Oct 11, 2023

Conversation

ringabout
Copy link
Member

Issues like nim-lang/nimble#1149 keep popping up. One way or another, we should alleviate the pain.

Finally, we should consider nim-lang/nimble#1141 (comment) as an option using some kind of cron script to update https://nim-lang.org/nimble/packages.json. It's turning into a really annoying problem.

Issues like nim-lang/nimble#1149 keep popping up. One way or another, we should alleviate the pain.
@Araq Araq merged commit d790112 into devel Oct 11, 2023
22 checks passed
@Araq Araq deleted the ringabout-patch-7 branch October 11, 2023 19:06
@github-actions
Copy link
Contributor

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

Hint: mm: orc; opt: speed; options: -d:release
174043 lines; 10.829s; 770.027MiB peakmem

@narimiran
Copy link
Member

I'm guessing this should also be backported to the 2.0.x line, right?

@metagn
Copy link
Collaborator

metagn commented Jan 6, 2024

The Nim boot stats above jump from 620 MB peakmem in #22813 and before to 770 MB here, the stats for every merged PR have hovered around this since. The boot times also seem to have gone from hitting 8 seconds pretty often to hitting around 10 on average now. Does this mean anything?

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.

None yet

4 participants