📝 手順書が警告どおりの事故を起こしうる形だったのを直す - #1903
Merged
Merged
Conversation
見直しで、手順書自身が禁じている操作を実行できる形になっていた。 ## コードブロックを丸ごと貼ると、警告した事故が起きる 「coderdojo.jp に出る前に DojoMap を起動すると、ジョブは成功するのに地図に 出ない」と警告しながら、その下の 3 ステップは独立したコマンドの並びだった。 丸ごと貼り付けると、1 段目が時間切れで終わっても 2 段目の gh workflow run が そのまま走る。 前の段階が成功しなければ次に進まないよう && でつないだ。未掲載の名前で ワークフローが起動しないことを実際に確認している。 ## 同じ確認を 2 度説明していた 3 段目で鞍手を使って地図を確認した直後に、南城を使って同じ確認を説明して いた。後者は「どのクラブと突合したか」を見る別の用途なので、そう書き分け、 例の Dojo 名も節内で統一した。 ## 既存の誤字 「省略化」を「省略可」に直した (2 箇所)。 ## 手順一覧の係り先 「地図も先に更新する」だけでは何より先か読み取れないため、 「メールを送る前に」を補った。
yasulab
force-pushed
the
fix-doc-guard-and-typos
branch
from
August 31, 2026 09:00
e6e8c47 to
9da60c3
Compare
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
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.
昨夜の #1901 / #1902 を通しで見直したところ、手順書自身が禁じている操作を実行できる形になっていました。
コードブロックを丸ごと貼ると、警告した事故が起きます
「coderdojo.jp に出る前に DojoMap を起動すると、ジョブは成功するのに地図に出ない」と警告しておきながら、その下の 3 ステップは独立したコマンドの並びでした。
丸ごと貼り付けると、1 段目が時間切れで終わっても 2 段目の
gh workflow runがそのまま走ります。前の段階が成功しなければ次に進まない形に直しました。
未掲載の名前でワークフローが起動しないことを、実際に動かして確認しています。
ブラウザから実行する場合も、先に coderdojo.jp 側を目視で確認するよう補いました。
同じ確認を 2 度説明していました
3 段目で
鞍手を使って地図を確認した直後に、南城を使って同じ確認を説明していました。別々の Dojo 名で同じことを 2 回説明する形です。後者は「どのクラブと突合したか」を見る別の用途なので、そう書き分け、例の Dojo 名も節内で統一しました(実データで出力例を取り直しています)。
既存の誤字
2 箇所ありました。
手順一覧の係り先
「先に」だけでは何より先か読み取れないため補いました。
検証
bash -nで構文を確認関連: map#45(地図側のドキュメントの取り残しを修正)