Skip to content

enhance the method of path.basename#421

Merged
mapleeit merged 1 commit intoform-data:masterfrom
xan-ok:master
Jun 21, 2019
Merged

enhance the method of path.basename#421
mapleeit merged 1 commit intoform-data:masterfrom
xan-ok:master

Conversation

@xan-ok
Copy link
Copy Markdown
Contributor

@xan-ok xan-ok commented Apr 12, 2019

value.client._httpMessage.path maybe return undefined, and cause error:
TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined
Add || '' to enhance quality。

value.client._httpMessage.path maybe return undefined, and cause error: 
`TypeError [ERR_INVALID_ARG_TYPE]: The "path" argument must be of type string. Received type undefined`。
Add `|| ''` to enhance quality。
@mapleeit
Copy link
Copy Markdown
Collaborator

LGTM

Maybe we should merge this. : ) @alexindigo

@alexindigo
Copy link
Copy Markdown
Member

Thank you @mapleeit, I added you as collaborator, please merge when you're ready.
I also retriggered appveyor.

@mapleeit
Copy link
Copy Markdown
Collaborator

Ok, thanks for the trust.


I saw CI rerun several times, but failed in different environment. A little weird.

@alexindigo
Copy link
Copy Markdown
Member

Yeah windows builds very unstable 🤦‍♂

@mapleeit mapleeit merged commit 87751e0 into form-data:master Jun 21, 2019
@mapleeit
Copy link
Copy Markdown
Collaborator

Yeah windows builds very unstable 🤦‍♂

🤭It really sucks. Is there any alternatives for windows building?

@codenirvana
Copy link
Copy Markdown

@mapleeit @alexindigo Yes Travis.
We were facing a similar issue with AppVeyor 😅

@mapleeit
Copy link
Copy Markdown
Collaborator

mapleeit commented Jun 24, 2019

@codenirvana Thanks for the information!

Could we use Travis for windows building instead? Considering the unstable cases now. @alexindigo

@alexindigo
Copy link
Copy Markdown
Member

alexindigo commented Jun 24, 2019

@MapleIT I wasn’t aware Travis has windows support. Do you have link to the docs by any chance? Thank you.

@alexindigo
Copy link
Copy Markdown
Member

Oh, I see it :) Yes, let’s try that thing. :)

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.

4 participants