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

Addressing performance findings #2172

Merged
merged 12 commits into from
Nov 15, 2024
Merged

Addressing performance findings #2172

merged 12 commits into from
Nov 15, 2024

Conversation

RobinTail
Copy link
Owner

@RobinTail RobinTail commented Nov 15, 2024

I wanna put all my recent performance findings into ESLint rules, so that I could address them globally later

  • Object.assign turned out to be significantly improved in Node 22, but it's still faster for extending existing aggregator in a reducer. For other cases spread is ok.

@RobinTail RobinTail added enhancement New feature or request miracle Mysterious events are happening here labels Nov 15, 2024
@RobinTail RobinTail added this to the v21 milestone Nov 15, 2024
@RobinTail RobinTail mentioned this pull request Nov 15, 2024
Copy link

coveralls-official bot commented Nov 15, 2024

Coverage Status

coverage: 100.0%. remained the same
when pulling 0e846af on collect-perf-findings
into dfd9d72 on make-v21.

@RobinTail
Copy link
Owner Author

should be enough so far

@RobinTail RobinTail marked this pull request as ready for review November 15, 2024 13:31
@RobinTail RobinTail force-pushed the collect-perf-findings branch from 62e322d to 46cf3c1 Compare November 15, 2024 15:30
@RobinTail RobinTail changed the title Collect performance findings Addressing performance findings Nov 15, 2024
Copy link
Owner Author

@RobinTail RobinTail left a comment

Choose a reason for hiding this comment

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

ok

@RobinTail RobinTail merged commit b288b3a into make-v21 Nov 15, 2024
11 checks passed
@RobinTail RobinTail deleted the collect-perf-findings branch November 15, 2024 15:37
RobinTail added a commit that referenced this pull request Nov 20, 2024
![Kesaria Abramidze,
37](https://github.com/user-attachments/assets/31eb16f8-1e7f-40d8-83cd-3b4602eb5922)

Kesaria Abramidze, 37 years young, one of Georgia's most well-known
transgender women, was stabbed to death in her flat in the capital,
Tbilisi, just a day after the country's parliament passed a major
anti-LGBT bill.

https://www.bbc.com/news/articles/cy0lnpn019xo

Transgender women suffer too frequently from transphobic violence and
cruelty, being the less protected social group. I'd like to raise an
awareness of this problem. Humans should be creators — not killers. But
most importantly, I want every transgender girl to have an opportunity
to create applications quickly and, in general, learn to write code
easily in order to receive job offers and leave dangerously transphobic
territories for more favorable and civilized ones, and live happily
there. Protect transgender women.

-----------------------------

This version aims to improve the security and the overall development
experience. In particular, the requirements for the Express version are
increased, and you can now run the secure server exclusively.
Alternative plural properties are replaced with universal singular ones
supporting arrays. Specifying the method when creating an endpoint is
now optional, which also makes it easier to assign endpoints on a same
route for different methods. Memory consumption reduced for Routing
traverse. Previously deprecated methods and properties have been
removed, several public interfaces changed in order to enable features
coming up later.

- #2083 
- #2087 
- #2086 
- #2100 
- #2122 
- #2139 
- #2148 
- #2128 
- #2162 
- #2167 
- #2168 
- #2172 
- #2176 
- #2175 
- #2185 
- #2187 
- #2192 
- #2193 
- #2194
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request miracle Mysterious events are happening here
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant