Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: switch out spectral for vacuum #46

Open
wants to merge 13 commits into
base: main
Choose a base branch
from
Open

chore: switch out spectral for vacuum #46

wants to merge 13 commits into from

Conversation

philsturgeon
Copy link
Collaborator

@philsturgeon philsturgeon commented Dec 30, 2024

Ditching Spectral (especially via the not ideal Super-Linter) to go with the entirely open-source (and loads quicker) vacuum CLI.

Blog post on integration incoming! 🥳

The JUnit integration is not perfect, but @daveshanley has been improving it using this as a test case. As of v0.15.1 it now shows line numbers, but we need to get the files reporting in the results too. Whenever that happens I'll update it, but this is good enough for now because we only have one file: openapi.yaml.

@philsturgeon philsturgeon force-pushed the vacuum branch 2 times, most recently from 4a4c6ed to fbbc877 Compare January 7, 2025 12:56
Copy link

github-actions bot commented Jan 7, 2025

TestsPassedSkippedFailed ❌️
API Lint Results13 ran0 passed0 skipped13 failed

@hack3rvaillant
Copy link

Hi @philsturgeon I discovered vacuum this morning. Seems awesome.

@daveshanley
Copy link

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.

3 participants