Skip to content

feat: support abort signals#30

Open
jGleitz wants to merge 1 commit intoskyhookadventure:masterfrom
jGleitz:feat/abort-signal
Open

feat: support abort signals#30
jGleitz wants to merge 1 commit intoskyhookadventure:masterfrom
jGleitz:feat/abort-signal

Conversation

@jGleitz
Copy link

@jGleitz jGleitz commented Apr 19, 2024

fixes #29

"eslint-plugin-import": "^2.22.1",
"eslint-plugin-prettier": "^3.1.4",
"jest": "^26.6.3",
"jest": "^29.7.0",
Copy link
Author

Choose a reason for hiding this comment

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

updating Jest was necessary to support AbortController in tests. At least v27 is required, I updated all the way.

"nock": "^13.0.5",
"prettier": "^2.2.1",
"ts-jest": "^26.4.4",
"ts-jest": "^29.1.2",
Copy link
Author

Choose a reason for hiding this comment

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

updating ts-jest was required to match the jest version

"@types/aws4": "^1.5.1",
"@types/jest": "^26.0.15",
"@types/node": "^14.14.10",
"@types/node": "^16.18.96",
Copy link
Author

Choose a reason for hiding this comment

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

updating @types/node was necessary to get the types for signal support. This should not change the minimum required node version of this project, as older versions of node will simply ignore the signal property.

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.

Support Abort Signals

1 participant