-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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 XO #7374
Conversation
fregante
commented
May 27, 2024
- Extracted from Update dependencies #7367
- Follows Update stylelint #7373
@@ -58,7 +58,7 @@ class FeatureFile { | |||
return existsSync(this.path); | |||
} | |||
|
|||
// eslint-disable-next-line n/prefer-global/buffer -- Type only | |||
// eslint-disable-next-line @typescript-eslint/ban-types -- Just passing it | |||
contents(): Buffer { |
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.
Buffer
is a Uint8Array
subclass, so you could just use
contents(): Buffer { | |
contents(): Uint8Array { |
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.
Buffer has an includes(string)
method but Uint8Array
doesn't. Since that's what I'm using it for, I'll keep Buffer for now so I can merge this. Happy to change it if there's a similar method that doesn't require conversions. Maybe later I'll just read it as utf-8 instead
This reverts commit b756861.