Skip to content

Commit 0b78483

Browse files
committed
chore: remove some eslint stragglers
1 parent 781e960 commit 0b78483

File tree

13 files changed

+0
-80
lines changed

13 files changed

+0
-80
lines changed

.eslintignore

Lines changed: 0 additions & 3 deletions
This file was deleted.

.eslintrc

Lines changed: 0 additions & 46 deletions
This file was deleted.

package-lock.json

Lines changed: 0 additions & 17 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,6 @@
8787
"devDependencies": {
8888
"@biomejs/biome": "^2.1.3",
8989
"@readme/standards": "^1.2.0",
90-
"@types/eslint": "^8.44.7",
9190
"@types/har-format": "^1.2.15",
9291
"@types/node": "^24.0.8",
9392
"@types/qs": "^6.9.10",

src/integration.test.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,6 @@ const inputFileNames = readdirSync(path.join(...expectedBasePath), 'utf-8');
7474

7575
const fixtures: [string, Request][] = inputFileNames.map(inputFileName => [
7676
inputFileName.replace(path.extname(inputFileName), ''),
77-
// eslint-disable-next-line import/no-dynamic-require, global-require, @typescript-eslint/no-require-imports
7877
require(path.resolve(...expectedBasePath, inputFileName)),
7978
]);
8079

@@ -126,26 +125,22 @@ const testFilter =
126125
* @link https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/eval#never_use_eval!
127126
*/
128127
function looseJSONParse(obj: any) {
129-
// eslint-disable-next-line no-new-func
130128
return new Function(`"use strict";return ${obj}`)();
131129
}
132130

133-
// eslint-disable-next-line @vitest/require-hook
134131
availableTargets()
135132
.filter(target => target.cli)
136133
.filter(testFilter('key', environmentFilter()))
137134
.forEach(target => {
138135
const { key: targetId, title, clients } = target;
139136

140137
describe.skipIf(process.env.NODE_ENV === 'test')(`${title} integration tests`, () => {
141-
// eslint-disable-next-line @vitest/require-hook
142138
clients.filter(testFilter('key', clientFilter(target.key))).forEach(({ key: clientId }) => {
143139
// If we're in an HTTPBin-powered Docker environment we only want to run tests for the
144140
// client that our Docker has been configured for.
145141
const shouldSkip = process.env.HTTPBIN && process.env.INTEGRATION_CLIENT !== targetId;
146142

147143
describe.skipIf(shouldSkip)(`${clientId}`, () => {
148-
// eslint-disable-next-line @vitest/require-hook
149144
fixtures.filter(testFilter(0, fixtureIgnoreFilter, true)).forEach(([fixture, request]) => {
150145
if (fixture === 'custom-method' && clientId === 'restsharp') {
151146
// restsharp doesn't even let you express calling an invalid

src/targets/index.test.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ const inputFileNames = readdirSync(path.join(...expectedBasePath), 'utf-8');
1717

1818
const fixtures: [string, Request][] = inputFileNames.map(inputFileName => [
1919
inputFileName.replace(path.extname(inputFileName), ''),
20-
// eslint-disable-next-line import/no-dynamic-require, global-require, @typescript-eslint/no-require-imports
2120
require(path.resolve(...expectedBasePath, inputFileName)),
2221
]);
2322

src/targets/javascript/fetch/client.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ export const fetch: Client<FetchOptions> = {
6666
}
6767

6868
// The FormData API automatically adds a `Content-Type` header for `multipart/form-data` content and if we add our own here data won't be correctly transmitted.
69-
// eslint-disable-next-line no-case-declarations -- We're only using `contentTypeHeader` within this block.
7069
const contentTypeHeader = getHeaderName(allHeaders, 'content-type');
7170
if (contentTypeHeader) {
7271
delete allHeaders[contentTypeHeader];

src/targets/json/native/client.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,6 @@ export const native: Client = {
4242
break;
4343
}
4444

45-
// eslint-disable-next-line no-case-declarations
4645
const multipartPayload: Record<string, any> = {};
4746
postData.params.forEach(param => {
4847
multipartPayload[param.name] = param.value;

src/targets/node/fetch/client.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@ export const fetch: Client = {
5757
}
5858

5959
// The FormData API automatically adds a `Content-Type` header for `multipart/form-data` content and if we add our own here data won't be correctly transmitted.
60-
// eslint-disable-next-line no-case-declarations -- We're only using `contentTypeHeader` within this block.
6160
const contentTypeHeader = getHeaderName(headersObj, 'content-type');
6261
if (contentTypeHeader) {
6362
delete headersObj[contentTypeHeader];

src/targets/php/helpers.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ export const convertType = (obj: any[] | any, indent?: string, lastIndent?: stri
2727

2828
case '[object Object]': {
2929
const result: string[] = [];
30-
// eslint-disable-next-line no-restricted-syntax
3130
for (const i in obj) {
3231
if (Object.prototype.hasOwnProperty.call(obj, i)) {
3332
result.push(`${convertType(i, indent)} => ${convertType(obj[i], `${indent}${indent}`, indent)}`);

0 commit comments

Comments
 (0)