search

ncaq / google-search-title-qualified

Google will omit the title of the web page. With this add-on, the original title is used as much as possible.
MIT License
16 stars 1 forks source link

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

Open ncaq opened 2 years ago

ncaq commented 2 years ago

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

ncaq commented 2 years ago

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

ncaq commented 1 year ago

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

ncaq commented 1 year ago

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

ncaq commented 1 year ago

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

ncaq commented 1 year ago

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

ncaq commented 1 year ago

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

ncaq commented 1 year ago

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

ncaq commented 1 year ago

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