cleanup: remove promise support check from tests by Phillip9587 · Pull Request #6148 · expressjs/express
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 22.6k
Comments
Merged
wesleytodd merged 1 commit intoexpressjs:masterfrom
Nov 12, 2024Merged
cleanup: remove promise support check from tests#6148
wesleytodd merged 1 commit intoexpressjs:masterfrom
cleanup: remove promise support check from tests#6148
wesleytodd merged 1 commit intoexpressjs:masterfrom
Conversation
Copy link
Member
Phillip9587
commented
Nov 5, 2024
Phillip9587
commented
Promises are supported in all supported Node.js version so the check is unnecessary
Promises are supported in all supported Node.js version so the check is unnecessary
bjohansebas approved these changes Nov 5, 2024
LinusU approved these changes Nov 5, 2024
UlisesGascon approved these changes Nov 7, 2024
wesleytodd approved these changes Nov 12, 2024
wesleytodd
merged commit
9e3dbb4
into
expressjs:master
Phillip9587
deleted the
remove-promise-check
branch
bjohansebas
mentioned this pull request
Closed
wesleytodd
mentioned this pull request
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment