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

yarn api-docs:lint does not work after #47 #78

Closed
Tracked by #17645
tnir opened this issue May 1, 2022 · 0 comments · Fixed by #80
Closed
Tracked by #17645

yarn api-docs:lint does not work after #47 #78

tnir opened this issue May 1, 2022 · 0 comments · Fixed by #80

Comments

@tnir
Copy link
Contributor

tnir commented May 1, 2022

Explain the changes

yarn api-docs:lint task, introduced in 130fde0 has failed after #47.

Suggested changes

Additional context

git checkout f6a7ec971162eea75160c121bf6d160b09377c74 && yarn run spectral lint -F hint -v api_v0.yml can reproduce it.

before #47 (f6a7ec9)

$ yarn run spectral lint -F hint -v api_v0.yml
yarn install v1.22.17
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[...]
[4/4] Building fresh packages...
Done in 9.78s.
yarn run v1.22.17
$ ~/node_modules/.bin/spectral lint -F hint -v api_v0.yml
Found 83 rules (67 enabled)
Linting ~/api_v0.yml
OpenAPI 3.x detected
No results with a severity of 'hint' or higher found!

Done in 2.81s.

after #47

$ yarn run spectral lint -F hint -v api_v0.yml
yarn install v1.22.17
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
[...]
[4/4] Building fresh packages...
Done in 10.35s.
yarn run v1.22.17
$ ~/node_modules/.bin/spectral lint -F hint -v api_v0.yml
Found 83 rules (67 enabled)
Linting ~/api_v0.yml
OpenAPI 3.x detected

~/api_v0.yml
 3466:11  warning  operation-tag-defined  Operation tags should be defined in global tags.  paths./admin/users.post.tags[0]

✖ 1 problem (0 errors, 1 warning, 0 infos, 0 hints)

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
tnir added a commit to tnir/forem-docs that referenced this issue May 1, 2022
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
tnir added a commit to tnir/forem-docs that referenced this issue May 1, 2022
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
tnir added a commit to tnir/forem-docs that referenced this issue May 1, 2022
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
tnir added a commit to tnir/forem-docs that referenced this issue May 1, 2022
Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
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 a pull request may close this issue.

1 participant