-
Notifications
You must be signed in to change notification settings - Fork 25
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
fix: replace node-fetch with axios #434
Conversation
use axios instead of note-fetch to better support proxy "re #285"
Jakob seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
use axios instead of note-fetch to better support proxy impemen https over http proxy support and fetch proxy settings from NPM "re #285"
fix: replace node-fetch with axios
This pull request introduces 1 alert when merging b1cb1e2 into e37d52c - view on LGTM.com new alerts:
|
use axios instead of note-fetch to better support proxy impemen https over http proxy support and fetch proxy settings from NPM "re #285"
fix: replace node-fetch with axios
This pull request introduces 1 alert when merging d3bbb89 into e37d52c - view on LGTM.com new alerts:
|
I've committed some code with another user appearantly. All tests are passing now, however they fail intermitently when running via the ci script. I think my node is struggling with the parallel processes. Would be good if you could take a look |
found a better solution |
use axios instead of note-fetch to better support proxy
"re #285"