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
chore: update repo #62
Conversation
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.
Thanks <3
Can you please check CI/lint issue? |
function genRawValue (value: unknown, indent = ""): string { | ||
if (typeof value === "undefined") { | ||
function genRawValue(value: unknown, indent = ""): string { | ||
if (typeof value === "undefined" || value === undefined) { |
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 added value === undefined
condition, othewise value
can't be type-narrowed at the point of value.toString()
.
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 don't know why, but TypeScript seems to not type-narrow value with typeof x === "undefined"
condition.
Since it passes unit tests, let me merge this PR for now.
But, please be aware that I included code change in this PR as well as applying the latest template. 🙌
I wanted to work on #24, but first I applied the latest unjs template for maintenance.