Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Oct 11, 2025

概要

voicevox_blogリポジトリのmasterブランチがプレビューページに表示されない問題を修正しました。

問題

https://voicevox.github.io/preview-pages/?repo=blog にアクセスした際、masterブランチのリンクが表示されていませんでした。

原因は、scripts/collectArtifacts.tsfetchTargets関数がmainブランチとproject-*ブランチのみをフィルタリングしており、masterブランチを含めていなかったためです。

変更内容

scripts/collectArtifacts.ts

ブランチフィルター条件にmasterブランチを追加しました:

const filteredBranches = branches.filter(
  (branch) =>
    branch.name.startsWith("project-") ||
    branch.name === "main" ||
    branch.name === "master",  // ← 追加
);

README.md

対象ブランチのリストにmasterを追加し、ドキュメントを実装と一致させました。

影響

  • mainブランチを使用しているリポジトリ: 影響なし(既存の動作を維持)
  • masterブランチを使用しているリポジトリ(voicevox_blog等): masterブランチがプレビューページに表示されるようになります
  • *project-ブランチ: 影響なし(既存の動作を維持)

関連Issue

Fixes #(Issue番号)

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • npm.jsr.io
    • Triggering command: node /usr/local/bin/pnpm install (dns block)
    • Triggering command: node /usr/local/bin/pnpm install --no-optional (dns block)
    • Triggering command: npm install (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>voicevox_blogのmasterブランチのリンクがない</issue_title>
<issue_description>## 不具合の内容

https://voicevox.github.io/preview-pages/?repo=blog
見た感じ、masterブランチのリンクがなさそうでした。

mainブランチはターゲットに入ってるけど、masterブランチは入ってないからっぽい・・・?

期待動作

masterブランチも表示される

その他

まあブランチ名をmasterではなくmainに変えるのが良い気もするけど、masterに対応するのでも・・・ありかもしれない・・・・・?
</issue_description>

<agent_instructions>おそらく scripts/collectArtifacts.ts を変えれば良いだけ。

PRなどは日本語で作ってください。
あとPRタイトルとかはconventional commit messageで。</agent_instructions>

Comments on the Issue (you are @copilot in this section)

Fixes #44


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Fix missing master branch link in voicevox_blog feat: masterブランチのサポートを追加 Oct 11, 2025
Copilot AI requested a review from Hiroshiba October 11, 2025 12:17
@Hiroshiba Hiroshiba marked this pull request as ready for review October 11, 2025 12:18
Copy link
Contributor

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 support for master branch to the preview pages system. The issue was that repositories using master branch (like voicevox_blog) were not appearing in the preview pages because the branch filtering logic only included main and project-* branches.

  • Added master branch to the branch filtering condition in fetchTargets function
  • Updated documentation to reflect the new supported branch types

Reviewed Changes

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

File Description
scripts/collectArtifacts.ts Modified branch filtering logic to include master branch alongside existing main and project-* patterns
README.md Added master to the list of supported target branches in documentation

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

ここ以外に変更箇所あるべきかわからないけど行けてそう感!

Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

お~ここまでできるんですね。すごい

@sevenc-nanashi
Copy link
Member

マージします。

@sevenc-nanashi sevenc-nanashi added this pull request to the merge queue Oct 11, 2025
Merged via the queue into main with commit 76b4f71 Oct 11, 2025
4 checks passed
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.

voicevox_blogのmasterブランチのリンクがない

3 participants