Skip to content

Conversation

@wclr
Copy link
Contributor

@wclr wclr commented Dec 13, 2021

Fixes #47

Formatted with default prettier settings.

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as I understand it, single quotes please!

this.cacheable();
}

const request = require.resolve("./to-array-buffer.js");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This repository appears to use single quotes only @wclr

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @M00TSDNS.

Thank you for your review, and I'm sorry for waiting you.
I will check the Pull Request today.

@pine
Copy link
Owner

pine commented Feb 16, 2023

Hello @wclr.
Thank you for your contribution. I'm sorry for waiting you long time.

I'm concerned about code style @M00TSDNS also said, but I will fix it after merging this.

@pine pine merged commit 2a92529 into pine:master Feb 16, 2023
@ghost
Copy link

ghost commented Feb 16, 2023

It's no worries Pine, hope life is treating you well!

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.

Won't work in browser's web worker

2 participants