Skip to content

fix: リソースリポジトリのバージョンを固定する#46

Merged
Hiroshiba merged 2 commits intomainfrom
fix/resource-version-pinning
Dec 20, 2025
Merged

fix: リソースリポジトリのバージョンを固定する#46
Hiroshiba merged 2 commits intomainfrom
fix/resource-version-pinning

Conversation

@Hiroshiba
Copy link
Member

@Hiroshiba Hiroshiba commented Dec 13, 2025

内容

リソースリポジトリの参照バージョンを固定できるようにしました。

env.json でresourceリポジトリのバージョンを定義できるようにしています。

関連 Issue

ref #43 のフェーズ1

スクリーンショット・動画など

N/A

その他

env.json をJSON形式にした理由は、シェルスクリプト・Python・GitHub Actionsのいずれからも読みやすい形式であるためです。

- env.jsonを追加し、参照するリソースリポジトリのバージョンを定義
- make_docs.pyがenv.jsonからバージョンを読み込むように変更
- MAINTENANCE.mdにenv.jsonについての説明を追加

close #43 のフェーズ1

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link

Copilot AI left a 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 adds the ability to pin resource repository versions by introducing a centralized env.json configuration file. The changes enable controlled version management of external VOICEVOX resource dependencies.

Key Changes:

  • Added env.json to define default resource repository versions (VOICEVOX and VOICEVOX Nemo)
  • Enhanced make_docs.py with argument parsing to accept resource references from env.json or command-line overrides
  • Updated documentation to reflect the new version management workflow

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
env.json Introduces version configuration with pinned versions "0.25.0" for both VOICEVOX resource repositories
scripts/make_docs.py Adds argument parsing, load_env() function, and updates URL construction to use configurable references instead of hardcoded "main" branch
docs/MAINTENANCE.md Updates maintenance documentation to include the new step of updating env.json before running scripts

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Hiroshiba
Copy link
Member Author

たぶん問題ないのでマージします!

@Hiroshiba Hiroshiba enabled auto-merge December 20, 2025 13:20
@Hiroshiba Hiroshiba added this pull request to the merge queue Dec 20, 2025
Merged via the queue into main with commit 64debf9 Dec 20, 2025
2 checks passed
@Hiroshiba Hiroshiba deleted the fix/resource-version-pinning branch December 20, 2025 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant