GitHub ActionsのUbuntuバージョンを変更 #276
Conversation
Walkthroughこの変更は、CI/CDワークフローとドキュメントにおいて、Ubuntu 20.04からUbuntu 24.04への移行を行い、関連するビルド設定やパス、READMEの記述を更新しています。また、C++ヘッダーのインクルード修正やサブモジュールのコミット更新も含まれています。 Changes
Sequence Diagram(s)sequenceDiagram
participant Developer
participant GitHub Actions
participant Ubuntu 24.04 Runner
Developer->>GitHub Actions: Push/PRをトリガー
GitHub Actions->>Ubuntu 24.04 Runner: ワークフロー実行（build/test/upload等）
Ubuntu 24.04 Runner-->>GitHub Actions: 結果を返す
GitHub Actions-->>Developer: ステータス通知
Poem
Tip ⚡️ Faster reviews with caching
Enjoy the performance boost—your workflow just got faster. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (8)
.github/actions/build/action.yml(0 hunks).github/workflows/build-and-test.yml(1 hunks).github/workflows/check-submodule-license.yml(1 hunks).github/workflows/upload-dlls-older-visual-studio.yml(4 hunks).github/workflows/upload-dlls.yml(3 hunks)3rdparty/glTF-SDK/glTF-SDK(1 hunks)README.md(1 hunks)include/plateau/dataset/city_model_package.h(1 hunks)
💤 Files with no reviewable changes (1)
- .github/actions/build/action.yml
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: build-and-test (windows-2022, x86_64)
- GitHub Check: build-and-test (ubuntu-24.04, x86_64)
- GitHub Check: build-and-test (macos-14, arm64)
🔇 Additional comments (12)
include/plateau/dataset/city_model_package.h (1)
4-4: 適切なヘッダ追加
uint32_tを利用するために<cstdint>をインクルードしており、ビルドエラーを回避しています。3rdparty/glTF-SDK/glTF-SDK (1)
1-1: サブモジュール glTF-SDK のコミット参照更新glTF-SDK サブモジュールのコミットを新しい
4888f0a386e174af6170c3c53b3396945f4a65a0に切り替えています。CI 環境更新と整合しており、依存関係の整合性確保として適切です。.github/workflows/check-submodule-license.yml (1)
18-18: Ubuntu ランナーを ubuntu-24.04 に更新
GitHub Actions がubuntu-24.04ランナーをサポートしているか公式ドキュメントでご確認ください。.github/workflows/build-and-test.yml (1)
27-27: マトリックスに ubuntu-24.04 を追加
Unity LTS 対応の環境に合わせた更新として妥当です。README.md (1)
132-132: Linux 推奨版を Ubuntu 24.04 に更新
手動ビルドで指定する Linux バージョンが正しく反映されています。.github/workflows/upload-dlls.yml (3)
26-26: Ubuntu ランナーを 24.04 に更新
ジョブマトリックスでubuntu-24.04が正しく設定されています。アーティファクト生成時のパス名とも一貫性があるかご確認を。
104-104: 動的ライブラリのパスを 24.04 に更新
~/a/libplateau-ubuntu-24.04/*.soへの変更が一貫して適用されています。
117-117: 静的ライブラリのパスを 24.04 に更新
~/a/libplateau-ubuntu-24.04/*.aへの変更が期待通りです。.github/workflows/upload-dlls-older-visual-studio.yml (4)
23-23: Ubuntu ランナーを 24.04 に更新
レガシーワークフローでもubuntu-24.04に統一されています。
46-46: Visual Studio バージョンを 17 (2022) に更新
アップロードアクションが VS2022 をサポートしているかご確認ください。
101-101: 動的ライブラリのパスを 24.04 に更新
Legacy ワークフローでも/libplateau-ubuntu-24.04/*.soが適用されています。
113-113: 静的ライブラリのパスを 24.04 に更新
Legacy ワークフローでも/libplateau-ubuntu-24.04/*.aへ変更済みです。
README.md
Outdated
| 利用する Linux は、Unityの対応OSに合わせて Ubuntu 20.04 とします。 | ||
| 利用する Linux は、Unityの対応OSに合わせて Ubuntu 24.04 とします。 | ||
| #### C++のビルド | ||
| * Ubuntu 20 はデフォルトでは git lfs がないので、`sudo apt install git-lfs` します。 |
There was a problem hiding this comment.
Ubuntu 20 の言及を 24.04 に更新
手動ビルド手順で「Ubuntu 20 はデフォルトでは git lfs がないので」とありますが、Ubuntu 24.04 に合わせて更新が必要です。
- * Ubuntu 20 はデフォルトでは git lfs がないので、`sudo apt install git-lfs` します。
+ * Ubuntu 24.04 はデフォルトでは git lfs がないので、`sudo apt install git-lfs` します。📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| * Ubuntu 20 はデフォルトでは git lfs がないので、`sudo apt install git-lfs` します。 | |
| * Ubuntu 24.04 はデフォルトでは git lfs がないので、`sudo apt install git-lfs` します。 |
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
134-134: Unordered list style
Expected: dash; Actual: asterisk
(MD004, ul-style)
There was a problem hiding this comment.
Actionable comments posted: 0
♻️ Duplicate comments (1)
README.md (1)
132-132: Ubuntuバージョンの更新: 24.04への置換確認
Ubuntu 20.04の言及がUbuntu 24.04に正しく更新されています。
🧹 Nitpick comments (1)
README.md (1)
134-134: 箇条書きスタイルの一貫性
リストの箇条書きにおいて、他の箇条書き（ダッシュ）とスタイルを統一するため、アスタリスク（*）ではなくダッシュ（-）を使用してください。- * Ubuntuでgit lfsを使うために`sudo apt install git-lfs` します。 + - Ubuntuでgit lfsを使うために`sudo apt install git-lfs` します。🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
134-134: Unordered list style
Expected: dash; Actual: asterisk(MD004, ul-style)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
🪛 markdownlint-cli2 (0.17.2)
README.md
134-134: Unordered list style
Expected: dash; Actual: asterisk
(MD004, ul-style)
⏰ Context from checks skipped due to timeout of 90000ms (3)
- GitHub Check: build-and-test (ubuntu-24.04, x86_64)
- GitHub Check: build-and-test (macos-14, arm64)
- GitHub Check: build-and-test (windows-2022, x86_64)
前：GitHub ActionsのUbuntuランナーのバージョンが古いためにCIが落ちていた。
今：Ubuntuバージョンを更新して落ちないようにした。
Summary by CodeRabbit
ドキュメント
チョア
リファクタ