search

expressjs / express

Fast, unopinionated, minimalist web framework for node.
https://expressjs.com
MIT License
65.73k stars 16.35k forks source link

cleanup: remove AsyncLocalStorage check #6147

Closed Phillip9587 closed 1 week ago

Phillip9587 commented 2 weeks ago

with the minimum supported node version set to v18 we can remove the AsyncLocalStorage support check as it was added in v13.10.0 and v12.17.0

wesleytodd commented 1 week ago

@Phillip9587 I realized I should have said this when I added my approval, but until we automate the history.md updates, can we continue manually adding things when they are code changes? I think we should have a discussion if test changes like this should have it, but I would rather folks have more in there than less until we automate or decide this is not necessary anymore.

Phillip9587 commented 1 week ago

@wesleytodd I added the HISTORY.md entry.