Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fallback to abort-controller polyfill #162

Merged
merged 2 commits into from
Jan 4, 2025

Conversation

Birowsky
Copy link
Contributor

@Birowsky Birowsky commented Jan 4, 2025

The abort-controller package doesn't really act as a polyfill to AbortController. Therefore, we're falling back to the package if the AbortController is not available

@ospfranco
Copy link
Collaborator

I think the tests are broken on CI due to bot detections. Did you run the tests locally?

@Birowsky
Copy link
Contributor Author

Birowsky commented Jan 4, 2025

I just did, sorry. Here's what I got:

image

@ospfranco
Copy link
Collaborator

cool thanks!

@ospfranco ospfranco merged commit 11b14fc into OP-Engineering:main Jan 4, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants