camaraproject / EdgeCloud

Repository to describe, develop, document and test the EdgeCloud API family
Apache License 2.0
16 stars 43 forks source link

Linting workflow added #216

Closed Kevsy closed 6 months ago

Kevsy commented 7 months ago

Commonalities release 0.3.0 defines the linting integration guidelines for automated testing via Github actions. This PR adds the files and configuration to achieve this.

github-actions[bot] commented 7 months ago

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 2 0 0.03s
✅ JSON eslint-plugin-jsonc 1 0 0 1.41s
✅ JSON jsonlint 1 0 0.18s
✅ JSON prettier 1 1 0 0.94s
✅ JSON v8r 1 0 1.46s
❌ OPENAPI spectral 9 7 17.15s
✅ REPOSITORY git_diff yes no 0.77s
✅ REPOSITORY secretlint yes no 4.61s
❌ YAML yamllint 9 1 4.86s

See detailed report in MegaLinter reports

_MegaLinter is graciously provided by OX Security_

Kevsy commented 7 months ago

^ note the error above is the result of the linter running against the commit, i..e the linter files themselves.

nicolacdnll commented 6 months ago

I would like we get this PR merged sooner rather than later. Could we merge it?

What I see from the errors of the mega linter, most the issues are fixed by https://github.com/camaraproject/EdgeCloud/pull/222

Here a resume of the errors:

Kevsy commented 6 months ago

@nicolacdnll thanks and agree - you should be able to merge the PR (I can't merge because I created the PR)