Skip to content

Commit

Permalink
apply suggestions from code review
Browse files Browse the repository at this point in the history
  • Loading branch information
tsctx committed Apr 12, 2024
1 parent a16334d commit c76954a
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions benchmarks/fetch/isomorphic-encode.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,6 @@ function isomorphicEncode1 (input) {
return input
}

/**
* @see https://infra.spec.whatwg.org/#isomorphic-encode
* @param {string} input
*/
function isomorphicEncode2 (input) {
// 1. Assert: input contains no code points greater than U+00FF.
if (invalidIsomorphicEncodeValueRegex.test(input)) {
Expand All @@ -44,12 +40,13 @@ function isomorphicEncode2 (input) {
}

const settings = {
small: `${generateAsciiString(10)}`,
middle: `${generateAsciiString(30)}`,
long: `${generateAsciiString(70)}`
small: 10,
middle: 30,
long: 70
}

for (const [runName, value] of Object.entries(settings)) {
for (const [runName, length] of Object.entries(settings)) {
const value = generateAsciiString(length)
[

Check failure on line 50 in benchmarks/fetch/isomorphic-encode.mjs

View workflow job for this annotation

GitHub Actions / Lint

Expected indentation of 4 spaces but found 2

Check failure on line 50 in benchmarks/fetch/isomorphic-encode.mjs

View workflow job for this annotation

GitHub Actions / Lint

Unexpected newline between object and [ of property access
{ name: `${runName} (valid)`, value },

Check failure on line 51 in benchmarks/fetch/isomorphic-encode.mjs

View workflow job for this annotation

GitHub Actions / Lint

Expected indentation of 6 spaces but found 4

Check failure on line 51 in benchmarks/fetch/isomorphic-encode.mjs

View workflow job for this annotation

GitHub Actions / Lint

'value' was used before it was defined

Check failure on line 51 in benchmarks/fetch/isomorphic-encode.mjs

View workflow job for this annotation

GitHub Actions / Lint

Unexpected use of comma operator
{

Check failure on line 52 in benchmarks/fetch/isomorphic-encode.mjs

View workflow job for this annotation

GitHub Actions / Lint

Expected indentation of 6 spaces but found 4
Expand Down

0 comments on commit c76954a

Please sign in to comment.