Skip to content

GitHub Actions へ集約のために appveyor を削除 #2091

Merged
m-tmatma merged 1 commit into
sakura-editor:masterfrom
m-tmatma:feature/remove-appveyor2
Jun 24, 2025
Merged

GitHub Actions へ集約のために appveyor を削除 #2091
m-tmatma merged 1 commit into
sakura-editor:masterfrom
m-tmatma:feature/remove-appveyor2

Conversation

@m-tmatma

@m-tmatma m-tmatma commented Jun 24, 2025

Copy link
Copy Markdown
Member

PR対象

  • ビルド手順/CI

カテゴリ

  • 削除

PR の背景

#1809 で GitHub Actions への統一の方針があり、
#2089 で azure pipeline を削除した。
appveyor に関しても削除する。

仕様・動作説明

appveyor の JOB を削除
appveyor のすべての影響範囲の削除まではスコープ外

appveyor の設定で appveyor のビルドを OFF にした。
https://ci.appveyor.com/project/sakuraeditor/sakura/settings/build

image

https://ci.appveyor.com/project/sakuraeditor/sakura/settings/tests

image

PR の影響範囲

appveyor の JOB が走らなくなる。

テスト内容

本 PR において appveyor の JOB が実際には走ってない。(内部的に skip されている)

関連 issue, PR

#1809
#2089

参考資料

@AppVeyorBot

Copy link
Copy Markdown

Build sakura 1.0.4578 completed (commit 0537c9b61a by @m-tmatma)

@m-tmatma m-tmatma changed the title appveyor を削除 GitHub Actions へ集約のために appveyor を削除 Jun 24, 2025
@m-tmatma

Copy link
Copy Markdown
Member Author

Build sakura 1.0.4578 completed (commit 0537c9b61a by @m-tmatma)

プロジェクト設定を変更してから PR を作り直した。
個人アカウントで appveyor のプロジェクトを設定している人は
自分のアカウントのビルド設定を OFF にする必要がある。

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 aims to remove the Appveyor configuration as part of the migration to GitHub Actions.

  • Completely deletes the appveyor.yml file to disable the Appveyor builds.
Comments suppressed due to low confidence (1)

@m-tmatma m-tmatma added CI appveyor など CI 関連 【ChangeLog除外】 appveyor labels Jun 24, 2025
@berryzplus

Copy link
Copy Markdown
Contributor

CI/CDについての説明がマークダウンに書かれているので、どこかで更新する必要あり。

もともと整理されていたわけでもないので、Appveyor停止を目的としてこのPRは先行マージしてよさそうに思います。

@m-tmatma m-tmatma merged commit c1b7894 into sakura-editor:master Jun 24, 2025
25 checks passed
@m-tmatma

Copy link
Copy Markdown
Member Author

https://ci.appveyor.com/project/sakuraeditor/sakura/builds/52288603
は走っている。

sakuraeditor のプロジェクト自体を削除しないと走ってしまうのかな。

@m-tmatma

Copy link
Copy Markdown
Member Author

https://ci.appveyor.com/project/sakuraeditor/sakura/settings

Skip branches without appveyor.yml のチェックを入れてみた。

image

@m-tmatma m-tmatma deleted the feature/remove-appveyor2 branch June 27, 2025 03:54
@m-tmatma

Copy link
Copy Markdown
Member Author

image

以下のチェックも入れてみた。

Do not build on "Push" events ?
Do not build on "Pull request" events ?

@m-tmatma

Copy link
Copy Markdown
Member Author

#2093 で update branchボタン押したら
appveyorのチェックが消えた

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

appveyor CI appveyor など CI 関連 【ChangeLog除外】

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants