-
Notifications
You must be signed in to change notification settings - Fork 0
fix: make update-book-status workflow valid YAML #81
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR fixes critical YAML syntax errors in the update-book-status.yml GitHub Actions workflow that prevented the workflow from being parsed and executed. The changes address invalid heredoc placement, tab/space indentation inconsistencies, and multi-line string handling.
Key changes:
- Fixed heredoc block indentation for progress report generation (lines 105-120)
- Converted multi-line git commit message to multiple
-mflags (lines 146-148) - Replaced multi-line
--bodyargument with--body-fileapproach using a temporary file (lines 169-186)
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| cat > progress_report.md << EOF | ||
| # 📊 IT Engineer Knowledge Architecture 進捗レポート | ||
| # 📊 IT Engineer Knowledge Architecture 進捗レポート | ||
| **更新日時**: $(date '+%Y年%m月%d日 %H:%M:%S JST') | ||
| **更新日時**: $(date '+%Y年%m月%d日 %H:%M:%S JST') | ||
| ## 📈 全体統計 | ||
| ## 📈 全体統計 | ||
| - **総書籍数**: $total_books冊 | ||
| - **アクティブ書籍**: $active_books冊 | ||
| - **総Issue数**: $total_issues件 | ||
| - **総Star数**: $total_stars個 | ||
| - **総書籍数**: $total_books冊 | ||
| - **アクティブ書籍**: $active_books冊 | ||
| - **総Issue数**: $total_issues件 | ||
| - **総Star数**: $total_stars個 | ||
| ## 📚 書籍別ステータス | ||
| ## 📚 書籍別ステータス | ||
| | 書籍名 | ステータス | 最終更新 | Issues | Stars | | ||
| |--------|------------|----------|--------|-------| | ||
| EOF | ||
| | 書籍名 | ステータス | 最終更新 | Issues | Stars | | ||
| |--------|------------|----------|--------|-------| | ||
| EOF |
Copilot
AI
Dec 23, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The heredoc content is now indented with spaces. This indentation will be preserved in the generated progress_report.md file, causing all lines to have leading spaces. To maintain proper YAML indentation while avoiding leading spaces in the output file, use the <<-EOF syntax (with a dash) which strips leading tabs, or use <<EOF with unindented content.
Recommended approach: Keep the heredoc delimiter line indented but use <<-EOF with tab-indented content inside the heredoc, or restructure to avoid the indentation issue entirely by using echo statements instead of a heredoc.
update-book-status.ymlがYAMLとして不正で、GitHub Actions が読み込めない状態になっていたため修正しました。run: |ブロック内のヒアドキュメント/複数行文字列が未インデントでトップレベルに露出していた対応:
run: |配下に正しくインデントgit commit -mの複数行を-mの複数指定に変更gh issue create --bodyの複数行を--body-fileに変更(本文は一時ファイル生成)run内のタブをスペースに統一ワークフローの意図(生成物/内容)は変えず、構文健全性を回復する修正です。