Skip to content
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

Closed
wants to merge 9 commits into from
Closed

fix: replace node-fetch with axios #434

wants to merge 9 commits into from

Conversation

uxkjaer
Copy link
Member

@uxkjaer uxkjaer commented Nov 30, 2021

use axios instead of note-fetch to better support proxy

"re #285"

use axios instead of note-fetch to better support proxy

"re #285"
@cla-assistant
Copy link

cla-assistant bot commented Nov 30, 2021

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ uxkjaer
❌ Jakob


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.

uxkjaer and others added 2 commits December 1, 2021 14:37
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
@lgtm-com
Copy link

lgtm-com bot commented Dec 1, 2021

This pull request introduces 1 alert when merging b1cb1e2 into e37d52c - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

uxkjaer and others added 2 commits December 1, 2021 14:43
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
@lgtm-com
Copy link

lgtm-com bot commented Dec 1, 2021

This pull request introduces 1 alert when merging d3bbb89 into e37d52c - view on LGTM.com

new alerts:

  • 1 for Unused variable, import, function or class

@uxkjaer
Copy link
Member Author

uxkjaer commented Dec 1, 2021

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

@uxkjaer uxkjaer closed this Jan 7, 2022
@uxkjaer
Copy link
Member Author

uxkjaer commented Jan 7, 2022

found a better solution

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.

None yet

1 participant