Skip to content

Integrate lint#333

Open
serowri wants to merge 6 commits intomainfrom
integrate-lint
Open

Integrate lint#333
serowri wants to merge 6 commits intomainfrom
integrate-lint

Conversation

@serowri
Copy link
Contributor

@serowri serowri commented Mar 19, 2026

変更点

  • 削除したファイル: .yamllint
    • yamllint実行時にルール指定を行うように変更した。 (ルートディレクトリがごちゃごちゃするのはよくないと思った。)
  • 名称変更: lint.yml <- yamllint.yml
  • 以上2つより、.github/workflows以下のCIのpaths-ignoreを更新
  • 追加: actionlintをlint.ymlに追加

@serowri serowri self-assigned this Mar 19, 2026
@serowri serowri linked an issue Mar 19, 2026 that may be closed by this pull request
@serowri
Copy link
Contributor Author

serowri commented Mar 19, 2026

.github/workflows/wheel.yml:60:38: property "architecture" is not defined in object type {arch: string; cibw-os-arch: string; cibw-python: string; os: string; runs-on: string} [expression]
60 |           key: "${{ matrix.os }}-${{ matrix.architecture }}-${{ matrix.device }}"

https://github.com/qulacs/scaluq/actions/runs/23283229027/job/67700951452?pr=333#step:6:101
のように存在しないものにアクセスしているものがあったので、ひとまずmatrix.os-archに変更し、修正しています。

actionlintは$ENV"$ENV"と書かないと怒られたりで結構厳しいみたいです。
->infoを無効にしたのでこれについてはなくなりました。

warning, errorに関しては直すべきなのは当然ですが、infoについても解消しないとCIが失敗するようです。(厳しすぎたらreviewdogへの変更もありかもしれない。)

  • infoをignoreします。

@serowri serowri marked this pull request as ready for review March 19, 2026 07:37
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.

[Task] Integrate yamllint and actionlint

1 participant