Skip to content

Commit b594edd

Browse files
Uzlopakanonrig
authored andcommitted
update rome
1 parent f4ee455 commit b594edd

File tree

7 files changed

+25
-28
lines changed

7 files changed

+25
-28
lines changed

.github/workflows/linter.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
uses: actions/setup-node@v3
2828

2929
- name: Install dependencies
30-
run: npm install
30+
run: npm install --ignore-scripts
3131

3232
- name: Linter
3333
run: npm run format:ci

benchmark/bench-thread.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ switch (benchmark.type) {
2929
break;
3030
}
3131

32-
suite.on("cycle", (event) => {
33-
parentPort.postMessage(String(event.target));
34-
}).run();
32+
suite
33+
.on("cycle", (event) => {
34+
parentPort.postMessage(String(event.target));
35+
})
36+
.run();

benchmark/bench.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ const benchmarks = [
5959
{
6060
type: "stringify",
6161
name: "{ id: 1e+22 }",
62-
input: { id: 1e+22 },
62+
input: { id: 1e22 },
6363
},
6464
{
6565
type: "stringify",

lib/internals/querystring.js

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -76,10 +76,10 @@ function encodeString(str) {
7676
if (c < 0x800) {
7777
lastPos = ++i;
7878
out += utf16[c];
79-
} else if (c < 0xD800) {
79+
} else if (c < 0xd800) {
8080
lastPos = ++i;
8181
out += utf16[c];
82-
} else if (c < 0xE000) {
82+
} else if (c < 0xe000) {
8383
// Surrogate pair
8484

8585
// This branch should never happen because all URLSearchParams entries
@@ -89,13 +89,13 @@ function encodeString(str) {
8989
throw new Error("URI malformed");
9090
}
9191

92-
c = 0x10000 + (((c & 0x3FF) << 10) | (str.charCodeAt(i) & 0x3FF));
92+
c = 0x10000 + (((c & 0x3ff) << 10) | (str.charCodeAt(i) & 0x3ff));
9393
lastPos = ++i;
9494
out +=
95-
hexTable[0xF0 | (c >> 18)] +
96-
hexTable[0x80 | ((c >> 12) & 0x3F)] +
97-
hexTable[0x80 | ((c >> 6) & 0x3F)] +
98-
hexTable[0x80 | (c & 0x3F)];
95+
hexTable[0xf0 | (c >> 18)] +
96+
hexTable[0x80 | ((c >> 12) & 0x3f)] +
97+
hexTable[0x80 | ((c >> 6) & 0x3f)] +
98+
hexTable[0x80 | (c & 0x3f)];
9999
} else {
100100
if (lastPos < i) {
101101
out += str.slice(lastPos, i);

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
"query-string": "^7.1.1",
3939
"querystringify": "^2.2.0",
4040
"querystringparser": "^0.1.1",
41-
"rome": "0.9.1-next",
41+
"rome": "11.0.0",
4242
"simple-git": "^3.14.1",
4343
"vitest": "^0.23.4"
4444
},

test/parse.test.ts

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,18 +4,17 @@ import { qsNoMungeTestCases, qsTestCases, qsWeirdObjects } from "./node";
44
import querystring from "querystring";
55

66
test("should succeed on node.js tests", () => {
7-
qsWeirdObjects.forEach(
8-
(t) =>
9-
assert.deepEqual(qs.parse(t[1] as string), t[2] as Record<string, any>),
7+
qsWeirdObjects.forEach((t) =>
8+
assert.deepEqual(qs.parse(t[1] as string), t[2] as Record<string, any>),
109
);
1110
qsNoMungeTestCases.forEach((t) => assert.deepEqual(qs.parse(t[0]), t[1]));
1211
qsTestCases.forEach((t) => assert.deepEqual(qs.parse(t[0]), t[2]));
1312
});
1413

1514
test("native querystring module should match the test suite result", () => {
1615
qsTestCases.forEach((t) => assert.deepEqual(querystring.parse(t[0]), t[2]));
17-
qsNoMungeTestCases.forEach(
18-
(t) => assert.deepEqual(querystring.parse(t[0]), t[1]),
16+
qsNoMungeTestCases.forEach((t) =>
17+
assert.deepEqual(querystring.parse(t[0]), t[1]),
1918
);
2019
});
2120

test/stringify.test.ts

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -4,23 +4,19 @@ import { test, assert } from "vitest";
44
import querystring from "querystring";
55

66
test("should succeed on node.js tests", () => {
7-
qsWeirdObjects.forEach(
8-
(t) =>
9-
assert.deepEqual(
10-
qs.stringify(t[2] as Record<string, any>),
11-
t[1] as string,
12-
),
7+
qsWeirdObjects.forEach((t) =>
8+
assert.deepEqual(qs.stringify(t[2] as Record<string, any>), t[1] as string),
139
);
1410
qsNoMungeTestCases.forEach((t) => assert.deepEqual(qs.stringify(t[1]), t[0]));
1511
qsTestCases.forEach((t) => assert.deepEqual(qs.stringify(t[2]), t[1]));
1612
});
1713

1814
test("native querystring module should match the test suite result", () => {
19-
qsTestCases.forEach(
20-
(t) => assert.deepEqual(querystring.stringify(t[2]), t[1]),
15+
qsTestCases.forEach((t) =>
16+
assert.deepEqual(querystring.stringify(t[2]), t[1]),
2117
);
22-
qsNoMungeTestCases.forEach(
23-
(t) => assert.deepEqual(querystring.stringify(t[1]), t[0]),
18+
qsNoMungeTestCases.forEach((t) =>
19+
assert.deepEqual(querystring.stringify(t[1]), t[0]),
2420
);
2521
});
2622

0 commit comments

Comments
 (0)