Skip to content

Commit 39a51ad

Browse files
committed
fix(api-linter): fix typo according to copilot check
1 parent d64c703 commit 39a51ad

File tree

9 files changed

+13
-14
lines changed

9 files changed

+13
-14
lines changed

api-linter/README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@ Usage: api-linter -s "path-to-spec-file" -r "path-to-rule-file" [-f "fail-severi
5656
5757
Options:
5858
-s, --specPath <openapi spec> path to openapi specification
59-
-r, --rulesetpath <rule file> path to rules file
59+
-r, --rulesetPath <rule file> path to rules file
6060
-f, --failSeverity <fail severity> test fails when met result the severity equal or higher than it (choices: "error", "warn", "info", "hint", default: warn)
6161
-c, --consoleSeverity <console severity> console output message for linter result no matter job failed or succeeded (choices: "error", "warn", "info", "hint", default: warn)
6262
-o, --outputFilename <output filename> specify the output filename with or without .html extension (default: linter-result.html)
@@ -107,7 +107,7 @@ extends:
107107
- "@siemens/api-linter/rulesets/siemens-api-common-operation.yml"
108108
- "@siemens/api-linter/rulesets/siemens-api-security.yml"
109109
```
110-
> You MAY use as below to achieve this for guidline linting `WITHOUT` `"spectral:oas"`.
110+
> You MAY use as below to achieve this for guideline linting `WITHOUT` `"spectral:oas"`.
111111
```yaml
112112
extends:
113113
- "@siemens/api-linter/rulesets/siemens-api-express.yml"

api-linter/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "@siemens/api-linter",
33
"version": "0.8.4",
4-
"description": "A linter tool with Rulesets follow Siemens RestApi Guidelines",
4+
"description": "A linter tool with Rulesets follow Siemens REST API Guidelines",
55
"homepage": "https://github.com/siemens/lint",
66
"author": {
77
"name": "Siemens",

api-linter/rulesets/functions/is-problem-json-schema.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ const assertProblemSchema = (schema) => {
3333
}
3434
const status = (schema.properties || {}).status || {};
3535
if (status.type !== 'integer' || status.format !== 'int32') {
36-
throw "Problem json must have property 'status' with type 'integer' and format 'in32'";
36+
throw "Problem json must have property 'status' with type 'integer' and format 'int32'";
3737
}
3838
const detail = (schema.properties || {}).detail || {};
3939
if (detail.type !== 'string') {

api-linter/rulesets/functions/links-has-self.js

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ const assertLinkSchema = (schema) => {
1111
const paths = jp.paths(schema, '$..*..links.properties');
1212

1313
for (const path of paths) {
14-
const pathString = path.join('.');
1514
if (path.some(p => p === 'relationships')) {
1615
continue;
1716
}

api-linter/rulesets/functions/query-parameters-length.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ export default (targetValue, options) => {
1111
}
1212
if (queryParameterCount > options.max && checkField && !hasFilter){
1313
return [{message: "The query parameter 'filter' SHOULD be used to filter or query data. " +
14-
"(Lint is unable to determine whether the parameters used for filtering or not.) "
14+
"(Linter is unable to determine whether the parameters used for filtering or not.) "
1515
}];
1616
}
1717
};

api-linter/src/index.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ const formatJSON = (rules, results) => {
106106
break;
107107
}
108108
}
109-
})
109+
});
110110
let jsonResult = {
111111
"data": data,
112112
"timestamp": new Date().toLocaleString(),
@@ -205,7 +205,7 @@ const linter = async (specFilePath, rulesetPath, followRefs) => {
205205
fs.unlinkSync(tempFilePath);
206206
}
207207
} catch (cleanupErr) {
208-
console.error(`Remove temporary file ${tempFilePath} failed. Please delete mannually.`);
208+
console.error(`Remove temporary file ${tempFilePath} failed. Please delete manually.`);
209209
}
210210

211211
return binded;
@@ -269,7 +269,7 @@ const resolveRuleDependencyResult = (rules, results) => {
269269

270270
const filteredResults = [];
271271

272-
for (const [locationKey, issues] of issuesByLocation.entries()) {
272+
for (const [_, issues] of issuesByLocation.entries()) {
273273
const failedRules = new Set(issues.map(i => i.code));
274274

275275
for (const issue of issues) {

api-linter/test/api-errors-refs.test.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
// test/api-errors-refs.test.js
22
const fs = require('fs');
3-
const { Spectral, Document } = require('@stoplight/spectral-core');
3+
const { Document } = require('@stoplight/spectral-core');
44
const Parsers = require('@stoplight/spectral-parsers');
55

66
// 👇 RE-ADD setupSpectral here
7-
const { setupSpectral, retrieveDocument } =
7+
const { setupSpectral } =
88
require('@jamietanna/spectral-test-harness');
99

10-
const { resultsForSeverity, getSpecFilePath } = require('./base.js');
10+
const { resultsForSeverity } = require('./base.js');
1111
const { resolver } = require('@stoplight/spectral-ref-resolver');
1212
const path = require('path');
1313

api-linter/test/api-pagination.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const { retrieveDocument, setupSpectral} = require('@jamietanna/spectral-test-harness')
2-
const {resultsForSeverity, getSpecFilePath,assertOnlyErrors, assertOnlyWarnings, assertOnlyInfos, assertOnlyHints } = require("./base.js")
2+
const {resultsForSeverity, getSpecFilePath, assertOnlyInfos, assertOnlyHints } = require("./base.js")
33

44
const path = 'pagination/';
55
const rulesetPath = "test/testdata/pagination/.spectral.yml";

api-linter/test/api-sparse-fieldsets.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const { retrieveDocument, setupSpectral} = require('@jamietanna/spectral-test-harness')
2-
const {resultsForSeverity, getSpecFilePath, assertOnlyWarnings, assertOnlyInfos } = require("./base.js")
2+
const {resultsForSeverity, getSpecFilePath, assertOnlyInfos } = require("./base.js")
33

44
const path = 'fields/';
55
const rulesetPath = "test/testdata/fields/.spectral.yml";

0 commit comments

Comments
 (0)