Skip to content

make lint-yaml missed all yaml files under fixtures #27268

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

Closed
lunny opened this issue Sep 26, 2023 · 3 comments · Fixed by #27284
Closed

make lint-yaml missed all yaml files under fixtures #27268

lunny opened this issue Sep 26, 2023 · 3 comments · Fixed by #27284

Comments

@lunny
Copy link
Member

lunny commented Sep 26, 2023

Description

run --list-files, it will not list all the yaml files.

poetry run yamllint --list-files .
./.drone.yml
./.spectral.yaml
./.stylelintrc.yaml
./.golangci.yml
./.changelog.yml
./.yamllint.yaml
./.gitpod.yml
./.markdownlint.yaml
./.eslintrc.yaml
./options/label/Advanced.yaml
./docs/themes/gitea/i18n/zh-CN.yaml
./docs/themes/gitea/i18n/zh-TW.yaml
./docs/themes/gitea/i18n/pt-BR.yaml
./docs/themes/gitea/i18n/en-US.yaml
./docs/themes/gitea/i18n/nl-NL.yaml
./.github/FUNDING.yml
./.github/labeler.yml
./.github/stale.yml
./.github/actionlint.yaml
./.github/workflows/cron-translations.yml
./.github/workflows/pull-db-tests.yml
./.github/workflows/pull-e2e-tests.yml
./.github/workflows/pull-compliance.yml
./.github/workflows/files-changed.yml
./.github/workflows/cron-lock.yml
./.github/workflows/cron-licenses.yml
./.github/workflows/pull-labeler.yml
./.github/workflows/pull-docker-dryrun.yml
./.github/workflows/release-nightly.yml
./.github/ISSUE_TEMPLATE/bug-report.yaml
./.github/ISSUE_TEMPLATE/feature-request.yaml
./.github/ISSUE_TEMPLATE/config.yml
./.github/ISSUE_TEMPLATE/ui.bug-report.yaml
./snap/snapcraft.yaml
./web_src/js/components/.eslintrc.yaml
./modules/migration/file_format_testdata/issue_a.yml

Gitea Version

main

Can you reproduce the bug on the Gitea demo site?

Yes

Log Gist

No response

Screenshots

No response

Git Version

No response

Operating System

No response

How are you running Gitea?

dev env

Database

None

@JakobDev
Copy link
Contributor

yamllint is configured to ignore the fixtures, so this is expected

gitea/.yamllint.yaml

Lines 44 to 48 in d0012c8

ignore: |
.venv
node_modules
/models/fixtures
/models/migrations/fixtures

@silverwind
Copy link
Member

Could un-ignore them.

@lunny
Copy link
Member Author

lunny commented Sep 26, 2023

Could un-ignore them.

I will do that.

techknowlogick pushed a commit that referenced this issue Sep 26, 2023
GiteaBot pushed a commit to GiteaBot/gitea that referenced this issue Sep 26, 2023
silverwind pushed a commit that referenced this issue Sep 26, 2023
Backport #27284 by @lunny

Fix #27268

Co-authored-by: Lunny Xiao <xiaolunwen@gmail.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants