super-linterアップデート#311
Merged
KazuhiroM merged 2 commits intoThe-Japan-DataScientist-Society:masterfrom Mar 24, 2026
Merged
Conversation
Bumps [super-linter/super-linter](https://github.com/super-linter/super-linter) from 7.2.1 to 8.3.1. - [Release notes](https://github.com/super-linter/super-linter/releases) - [Changelog](https://github.com/super-linter/super-linter/blob/main/CHANGELOG.md) - [Commits](super-linter/super-linter@v7.2.1...v8.3.1) --- updated-dependencies: - dependency-name: super-linter/super-linter dependency-version: 8.3.1 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
super-linterのv7からv8へのアップグレードに伴う各種修正。
Workflowの修正:
- super-linter/super-linterアクションをSHAでピン留め
- persist-credentials: falseを追加
- BIOME_LINTを無効化
- zizmor: テンプレートインジェクション警告を修正 (dockle.yml, fix_env.yml)
- dependabot: cooldownを7日に設定
Linter設定の追加:
- .github/linters/.python-lint: ノートブック向けpylint設定を追加
- .github/linters/trivy.yaml: TRIVY設定を追加
- .github/linters/trivy-secret.yaml: .gitleaks.tomlの誤検知を抑制
Notebookの修正:
super-linter v8でJupyter notebook向けlinterが有効化されたため、
各linterのエラーをnotebookに対して修正した。
docker/work直下の問題ノートブックとdocker/work/answer内の解答ノートブックは
対応関係にあるため、両方に同じ修正を適用した。
- JUPYTER_NBQA_BLACK: シングルクォートをダブルクォートに統一するなど、
blackのコードスタイルに準拠するようフォーマットを修正
- JUPYTER_NBQA_FLAKE8:
- F401: 演習用として配置されている未使用インポートに # noqa: F401 を追加
- E402: SQLノートブックでは %load_ext sql マジックコマンドの後にインポートが
あるためE402が発生する。これを抑制するため # noqa: E402 を追加
- F821: ans_preprocess_knock_Python.ipynb で display が未インポートのまま
使用されているため from IPython.display import display を追加。
また、問題・解答ノートブック間でインポートを揃えるため、
preprocess_knock_Python.ipynb にも同インポートを # noqa: F401 付きで追加
- JUPYTER_NBQA_ISORT: インポートの順序がisortのルールに従っていないため修正
- JUPYTER_NBQA_PYLINT:
- W0109: dtype辞書でpostal_cdキーが重複していたバグを修正
- R1705/R1710: pct_group関数でreturnの後にelifを使用していたため、
elif を if に変更し、返り値の一貫性も修正
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
KazuhiroM
merged commit afbc01e
into
The-Japan-DataScientist-Society:master
4 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
CVE-2026-25761 対処のため、super-linterをアップデートします。
合わせて以下の修正を行っています。
Workflowの修正
Linter設定の追加
Notebookの修正
super-linter v8でJupyter notebook向けlinterが有効化されたため、各linterのエラーをnotebookに対して修正しました。
この際、docker/work直下の問題ノートブックとdocker/work/answer内の解答ノートブックは対応関係にあるため、両方に同じ修正を適用しました。
JUPYTER_NBQA_BLACK
JUPYTER_NBQA_FLAKE8
また、問題・解答ノートブック間でインポートを揃えるため、preprocess_knock_Python.ipynb にも同インポートを # noqa: F401 付きで追加
JUPYTER_NBQA_ISORT
JUPYTER_NBQA_PYLINT