-
Notifications
You must be signed in to change notification settings - Fork 163
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
サクラエディタのデイリービルドを GitHub Release にアップできないか検討する #516
Comments
universal-ctags では https://github.com/universal-ctags/ctags ではなく |
元々これは、vim/vim-win32-installer で作った仕組みで、cron を走らせるシステムと、ビルドをするシステムは分けていたので、このような2段階になっていますが、本当は分ける必要はないはずです。 あと、GitHub Releasesにアップロードする際に使っているアクセストークンは専用アカウントを1つ用意しています。というのは、アクセストークンはアカウントに属するものなので、悪用されると同じアカウントの他のリポジトリに影響が出る可能性を危惧したからです。もっとも、アクセストークンは暗号化しているので、悪用されうるのは、同じリポジトリに書き込みできる誰かが悪意を持ったコミットを行った場合に限られるはずですが。 |
sakuraのデイリービルドリポジトリを用意し、正式タグ有りのビルドの場合はsakura本体のリポジトリにアップロードするというのは有りでしょうね。 |
draftの方がいいかも。draftだと、権限のある人しか見れません。 https://www.appveyor.com/docs/deployment/github/#provider-settings |
他のリポジトリを見てると、Deployする流れは主に3パターンあるみたいです。 ① gitでtagを手動で付けて、AppVeyorにはそれを見て動いてもらう方法。 「発行するバージョンをどこで管理(決定)するか」や「デプロイタイミング」で、これらを使い分ける感じだと思います。 ※結局どれもgitでtag付けが必要になるので、私は①が一番シンプルになると考えてます。(実際、②と③をやるとなると、appveyor.ymlの記載が多くなり、appveyor.ymlのテストも面倒です)
できるはずです。Personal access tokensの登録画面には、「どこからUPされるか」についての登録はありません。そのため、その正しいトークンが使われれば、どこからでもUPできると思います。
このあたりも参照ください。GitHub ReleasesとBinTrayの例を挙げてますが、2か所の異なるGitHub Releasesが対象の場合も同じです。(長文セット!) ところで、正式ビルドをGitHub Releasesにするのは、私も同意です。ですが、正式ビルド以外(デイリービルド等)までGitHub Releasesにこだわっている理由は何かあるんでしょうか? |
Dailyビルドが必要なほど更新されていないので一旦閉じてしまいます。 個人的な感覚では、1週間あたり2PRくらいマージされていく状況でなければDailyビルドは不要だと思います。 |
#481 で @jakenjarvis さんから 成果物を残す要望が上がっている
GitHub Release に正式版をあげるつもりではあるが、手動でアップすることを想定していた。
https://github.com/sakura-editor/sakura/releases にあげるのを検討する前に
実験もかねて appveyor から自動的に、https://github.com/sakura-editor/sakura
とは別にデイリービルドのアップ用のリポジトリを用意してそちらにアップロードしてみる。
他プロジェクトの例
The text was updated successfully, but these errors were encountered: