-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
ci: add windows #1326
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
base: main
Are you sure you want to change the base?
ci: add windows #1326
Conversation
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Signed-off-by: Sebastian Beltran <bjohansebas@gmail.com>
@@ -20,7 +20,7 @@ | |||
], | |||
"dependencies": { | |||
"append-field": "^1.0.0", | |||
"busboy": "^1.0.0", | |||
"@fastify/busboy": "^1.0.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Error Handling
should gracefully handle more than one error at a time:
Uncaught AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:
- actual - expected
- 'LIMIT_FILE_COUNT' - 'LIMIT_FILE_SIZE'
Test are failed because of changes in busboy
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, actually that change shouldn't be here. I was testing some things and accidentally committed it here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
fixed #1035