Closed Kevsy closed 2 months ago
Descriptor | Linter | Files | Fixed | Errors | Elapsed time |
---|---|---|---|---|---|
β ACTION | actionlint | 2 | 0 | 0.04s | |
β JSON | eslint-plugin-jsonc | 1 | 0 | 0 | 1.41s |
β JSON | jsonlint | 1 | 0 | 0.18s | |
β JSON | prettier | 1 | 1 | 0 | 0.96s |
β JSON | v8r | 1 | 0 | 2.41s | |
β OPENAPI | spectral | 9 | 6 | 16.86s | |
β REPOSITORY | git_diff | yes | no | 0.5s | |
β REPOSITORY | secretlint | yes | no | 4.47s | |
β YAML | yamllint | 9 | 1 | 4.15s |
See detailed report in MegaLinter reports
^ the linter error shown above relates to the other repository files. See first post for the successful Linter result for the Simple Edge Discovery YAML in isolation.
I see your explanation in commonalities
By convention HTTP headers are Kebab-Case: RFC 2616, section 5.3. So
IPv4-Address
uses the same naming style because it is an HTTP Header. "DeviceIpv4Address" is the schema that the parameter is based on (which follows CAMARA UpperCamelCase convention for schema names).
So for point number 2, it's clear to me.
Thanks @javierlozallu for your checking!
My mistake π€¦ββοΈ , I've made a new commit with the X-Correlator added to the request and responses.
I linted again locally with the following result:
kev@mac lint % spectral lint ../EdgeCloud/code/API_definitions/Discovery/simple_edge_discovery.yaml --verbose --ruleset .spectral.yml > eds3.txt
(node:67731) [DEP0040] DeprecationWarning: The `punycode` module is deprecated. Please use a userland alternative instead.
(Use `node --trace-deprecation ...` to show where the warning was created)
kev@mac lint % cat eds3.txt
Found 68 rules (58 enabled)
Linting /Users/kev/Code/Camara/EdgeCloud/code/API_definitions/Discovery/simple_edge_discovery.yaml
No results with a severity of 'error' found!%
Thanks
@Kevsy I have a comment for:
- simple-edge-discovery-read-closest
I'm adding 3Leg in LCM API and I found in Commonalities-API-design-guidelines that we should follow the structure:
Define a scope per API operation with the structure: api-name:[resource:]action
And for GET methods the action is:
read: For operations accessing to details of the resource, typically GET.
So I think that this change is needed:- simple-edge-discovery:edge-cloud-zones:read
Let me know what you think
Thanks @javierlozallu - I agree and I have committed the change to the PR.
Lint results
Found 68 rules (58 enabled)
Linting /Users/kev/Code/Camara/EdgeCloud/code/API_definitions/Discovery/simple_edge_discovery.yaml
No results with a severity of 'error' found!%
π
Changed basepath version to /v1 Changed API version number to 1.0.0
What type of PR is this?
Add one of the following kinds:
What this PR does / why we need it:
PR to propose Simple Edge Discovery 1.0.0 (ref minutes of Edge Cloud call 16 April )
Which issue(s) this PR fixes:
Updates version number to 1.0.0 and basepath to /v1
Special notes for reviewers:
Linter output for this file in isolation:
Changelog input