-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update: Changing on Dependencies #4
Changes from 6 commits
47e9170
26037a2
25cf4ed
d5e3b91
5dc2158
aab56c8
4d02acf
c2659ac
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -51,10 +51,10 @@ | |
}, | ||
"license": "MIT", | ||
"devDependencies": { | ||
"@typescript-eslint/eslint-plugin": "^6.11.0", | ||
"@typescript-eslint/parser": "^6.11.0", | ||
"bun-types": "latest", | ||
"eslint": "^8.53.0", | ||
"@typescript-eslint/eslint-plugin": "^6.16.0", | ||
"@typescript-eslint/parser": "^6.16.0", | ||
"bun-types": "^1.0.20", | ||
"eslint": "^8.55.0", | ||
"eslint-plugin-import": "^2.29.1" | ||
}, | ||
"peerDependencies": { | ||
|
@@ -63,6 +63,6 @@ | |
"dependencies": { | ||
"fast-querystring": "^1.1.2", | ||
"radix3": "^1.1.0", | ||
"reflect-metadata": "^0.1.13" | ||
"reflect-metadata": "^0.2.1" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. why we are changing dependencies is there any specific reason |
||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -280,7 +280,7 @@ export class Context { | |
this.res = new Response(null, { | ||
status: status, | ||
headers: { | ||
...this.res.headers.toJSON(), | ||
...Object.fromEntries(this.res.headers), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. It may be, but that usage fails pipeline for TS checking. I didn't prefer that, i have changed when pipeline was failed. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. interesting should't be fail can you rollback There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have added back and pipeline is failed again. Could you look that? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. just rollback all |
||
}, | ||
}); | ||
|
||
|
@@ -321,7 +321,7 @@ export class Context { | |
this.res = new Response(jsonData, { | ||
status: status || this.res.status, | ||
headers: { | ||
...this.res.headers.toJSON(), | ||
...Object.fromEntries(this.res.headers), | ||
"Content-Type": "application/json", | ||
Comment on lines
-324
to
325
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same above |
||
}, | ||
}); | ||
|
@@ -371,7 +371,7 @@ export class Context { | |
public getResWithoutBody(): Response { | ||
return new Response(null, { | ||
status: this.res.status, | ||
headers: this.res.headers, | ||
headers: Object.fromEntries(this.res.headers), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same above |
||
}); | ||
} | ||
|
||
|
@@ -400,7 +400,7 @@ export class Context { | |
this.res = new Response(data, { | ||
status: status || this.res.status, | ||
headers: { | ||
...this.res.headers.toJSON(), | ||
...Object.fromEntries(this.res.headers), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same above |
||
"Content-Type": contentType, | ||
}, | ||
}); | ||
|
@@ -426,7 +426,7 @@ export class Context { | |
this.res = new Response(data, { | ||
status: status || this.res.status, | ||
headers: { | ||
...this.res.headers.toJSON(), | ||
...Object.fromEntries(this.res.headers), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same above |
||
"Content-Type": contentType, | ||
}, | ||
}); | ||
|
@@ -452,7 +452,7 @@ export class Context { | |
this.res = new Response(data, { | ||
status: status || this.res.status, | ||
headers: { | ||
...this.res.headers.toJSON(), | ||
...Object.fromEntries(this.res.headers), | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. same above |
||
"Content-Type": contentType, | ||
}, | ||
}); | ||
|
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.
why we are changing dependencies is there any specific reason
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.
Just staying up to date. When we run bun update, it updates directly. Just i have removed latest tag from there because in the future it may create an issue.
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.
i guess
latest
can be stay forbun-types