Skip to content

Extend the api guidelines with a concrete ruleset.#7

Open
ravindrapalaskar17 wants to merge 672 commits intoExtend-the-API-guidelines-with-a-concrete-rulesetfrom
draft/Extend-the-API-guidelines-with-a-concrete-ruleset
Open

Extend the api guidelines with a concrete ruleset.#7
ravindrapalaskar17 wants to merge 672 commits intoExtend-the-API-guidelines-with-a-concrete-rulesetfrom
draft/Extend-the-API-guidelines-with-a-concrete-ruleset

Conversation

@ravindrapalaskar17
Copy link
Copy Markdown
Owner

@ravindrapalaskar17 ravindrapalaskar17 commented Oct 11, 2023

This is Internal PR for the base work of: -Extend the api guidelines with a concrete ruleset.

What type of PR is this?
enhancement/feature

What this PR does / why we need it:

  1. Added Spectral Linting into megaliter.
  2. Define spectral ruleset in .spectral.yaml
  3. Tried Spell-checking function of JavaScript in to spectral linting.
  4. created separate file qod-api2.yaml and checked JavaScript spell check function separately on this yaml file.
  5. Applied basic in-built spectral rules with --> extends: - "spectral:oas" for our api specification.

Which issue(s) this PR fixes:
Fixes camaraproject#15

Special notes for reviewers:
Decision outstanding whether spectral or Megalinter should be used.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 11, 2023

🦙 MegaLinter status: ❌ ERROR

Descriptor Linter Files Fixed Errors Elapsed time
✅ ACTION actionlint 2 0 0.03s
✅ JAVA checkstyle 3 0 3.84s
❌ OPENAPI spectral 2 1 3.38s
✅ REPOSITORY git_diff yes no 0.06s
✅ REPOSITORY grype yes no 11.56s
✅ REPOSITORY secretlint yes no 1.08s
✅ REPOSITORY trivy-sbom yes no 1.58s
✅ REPOSITORY trufflehog yes no 7.66s
✅ XML xmllint 3 0 0 0.07s
✅ YAML v8r 8 0 5.06s
❌ YAML yamllint 8 1 0.7s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

@ravindrapalaskar17 ravindrapalaskar17 self-assigned this Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant