make lint fails on shellcheck style and info warnings
まだ誰も着手していません。
評価
調査の方向性
make lintを実行してShellCheckの失敗を再現し、その後、gh-md-tocで報告された行を調べます。SC2006およびSC2181のスタイル警告に対処し、意図的なawk式は記載どおりに扱います。make lintがShellCheck 0.11.0で正常終了すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Steps
make lint
Actual
shellcheck -e SC2008 gh-md-toc exits with code 1 on master (c386c7b, shellcheck 0.11.0):
- SC2006 (style): legacy backticks instead of
$(...)- lines 170, 187, 246, 305-308, 371 - SC2181 (style): exit code checked indirectly with
$?- line 89 - SC2016 (info): expressions in single quotes - lines 249, 258
Expected
make lint passes.
Notes
- CI (
ci.yml) runs onlymake test, so the failure is not visible there. - Lines 249 and 258 start the awk scripts, which are single-quoted on purpose so that
$0reaches awk unexpanded. SC2016 there likely needs a disable directive rather
than a code change.
- 主要言語
- Shell
- スター
- 3.3k
- フォーク
- 2.7k
- 平均マージ
- 2分
- マージ済み PR(30日)
- 1
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドなし
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
ekalinin/github-markdown-toc のほかの issue
-
documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
ekalinin/github-markdown-toc#165 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
ekalinin/github-markdown-toc#166 ·
-
[Feature] `--no-code-tag-removal` flag allowing to make removal of backticks in TOC entries optionalオープンenhancement
難易度 3/5 1〜2日 初心者へのやさしさ 58/100
ekalinin/github-markdown-toc#153 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
ekalinin/github-markdown-toc#152 · リアクション 1 件 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
ekalinin/github-markdown-toc#141 ·
ekalinin/github-markdown-toc の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
YosysHQ/oss-cad-suite-build#216 ·
-
out-of-date
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
CachyOS/CachyOS-PKGBUILDS#1917 ·
メンテナーはふだん 1 日以内に返信
-
package-update
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
bug needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
EmbarkStudios/cargo-about#323 ·