Skip to content
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.

Sign up for GitHub

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

Jump to bottom

バージョンタグ付けたらCDでMozilla Firefoxアドオンストアに自動でデプロイされるようにする #86

Open
ncaq opened this issue Mar 7, 2022 · 8 comments
Open

バージョンタグ付けたらCDでMozilla Firefoxアドオンストアに自動でデプロイされるようにする #86

ncaq opened this issue Mar 7, 2022 · 8 comments
Labels
Priority: High Status: Need More Info Lacks enough info to make progress Type: CI Changes to CI configuration files and scripts Type: Release Related to release process

Comments

@ncaq
Copy link
Owner

ncaq commented Mar 7, 2022

avi12/web-ext-deploy: A tool for deploying WebExtensions to multiple stores.
を使用して、CircleCIによる自動デプロイをやめる。
web-ext-submitと違ってソースコードを提出できるのでMozillaに怒られないだろう。
@ncaq
Copy link
Owner Author

ncaq commented Mar 8, 2022

懸念点として、web-ext-deployはローカルのignoreしたファイルに秘密鍵を入れてローカルでデプロイすることを前提にしていて、CI/CDサポートがよく分からないことがある。
よって別の環境で作業すると鍵が分からないからデプロイ出来ない危険性が危惧される。
CDでの実行ならばタグを付けるだけなので問題ない。

ncaq added a commit that referenced this issue Mar 8, 2022
@ncaq
deleted: CircleCI: deploy
8fe2d31 
[web-ext-deployを使用する · Issue #86 · ncaq/google-search-title-qualified](#86)
をする前にとりあえずCDを削除する。
現状だとソース提出してなくてMozillaに怒られるだけなので。
@ncaq ncaq mentioned this issue Mar 8, 2022
Merged
@ncaq
Copy link
Owner Author

ncaq commented Jan 17, 2023

CLI実行でsecret入れれば良さそう

@ncaq
Copy link
Owner Author

ncaq commented Jan 28, 2023

もしかしてCDにこだわらずにリリース作業ぐらい手元でコマンド実行すれば良いのでは
webコンソールよりはよほどマシだし

@ncaq
Copy link
Owner Author

ncaq commented Feb 8, 2023

鍵についてはJavaScript実行にして、内部で環境変数を読み出すなりなんなりすれば特には問題無さそう。
問題はFirefoxそのままぶちこんだらFirefox for Android対応になってしまう可能性が高い。
現在テストしてないし、
通信料が増加するこれをwifi接続済みか判別するAPIがFirefoxに実装されてないので対応する予定もない。
何故バージョンの問題なのにmanifest.jsonで記述できないんだ?

@ncaq ncaq moved this to Todo in @ncaq TODO Feb 13, 2023
@ncaq ncaq added this to @ncaq TODO Feb 13, 2023
@ncaq ncaq moved this from Todo to Later in @ncaq TODO Feb 13, 2023
@ncaq
Copy link
Owner Author

ncaq commented Feb 16, 2023

とりあえずChromeだけ自動で対応というのもアリではある

@ncaq ncaq mentioned this issue Feb 16, 2023
Open
@ncaq
Copy link
Owner Author

ncaq commented Feb 23, 2023

web-extでsubmitした時自動的にAndroidでも有効になってしまうことを指摘
APIの方ではAndroid有効になってしまう問題解決できてるらしい。

@ncaq ncaq changed the title web-ext-deployを使用する バージョンタグ付けたらCDで自動でデプロイされるようにする Feb 23, 2023
@ncaq
Copy link
Owner Author

ncaq commented Feb 23, 2023

web-ext-deployはスクレイピング方式だし、web-extのsubmission apiを使って自分でどうにかする方が良さそう。
metadataは適当に互換性のやつを突っ込んでもなんとかなるのでは。

@ncaq ncaq changed the title バージョンタグ付けたらCDで自動でデプロイされるようにする バージョンタグ付けたらCDでMozilla Firefoxアドオンストアに自動でデプロイされるようにする Mar 24, 2023
@ncaq
Copy link
Owner Author

ncaq commented Mar 24, 2023

今の所実装が見つからないし、自分でActions書くのが一番楽という説すら出てきた

@ncaq ncaq added Estimate: 8 Type: CI Changes to CI configuration files and scripts Status: Need More Info Lacks enough info to make progress Type: Meta Type: Meta - Related to repository itself Type: Release Related to release process Priority: High and removed Estimate: 4 Priority: Medium Type: Maintenance Repository Maintenance Type: Meta Type: Meta - Related to repository itself labels Mar 24, 2023
@ncaq ncaq mentioned this issue Mar 24, 2023
Open
@ncaq ncaq mentioned this issue Mar 24, 2023
Open
@ncaq ncaq removed the Estimate: 8 label Oct 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Priority: High Status: Need More Info Lacks enough info to make progress Type: CI Changes to CI configuration files and scripts Type: Release Related to release process
Projects
@ncaq TODO
Status: TODO
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ncaq