【@kurudrive 確認待ち】無料版デプロイ時 incなどのフォルダも削除してからコミットしたい #1381
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
チケットへのリンク / 変更の理由(元のissueがあればリンクを貼り付ければOK)
実際に無料版にデプロイしないと確認が取れないのでチェック項目にチェックは入れていません。
背景
incなどのフォルダでは無料版リポジトリーに自動で削除されないので改めてブランチを作らなければいけない
新たにプルリクを作るのは面倒、また忘れる可能性が高いので自動で削除するようにしたい
例:vektor-inc/vk-blocks#119
srcフォルダ内はファイルの削除がされていた
例:吹き出しブロック
#1346
vektor-inc/vk-blocks@0812b9b
これはpro版から無料版へpushする際に一度無料版のsrcを削除しているためファイルの削除も反映されている
https://github.com/vektor-inc/vk-blocks-pro/blob/master/bin/deploy-free.sh#L26-L27
どういう変更をしたか?
editor-css/* inc/* lib/* options-css/* src/*
のフォルダ以下も削除するように変更※注意
全てを削除 = rm -rf * した方が全てのファイルを削除するので良いのでは?と思うかもしれないが
.gitignoreなどは無料版とpro版では違うものを使いたいっぽいので全ての削除はしない方が良いのかなと思っています
実装者の確認事項
実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。
プログラムの変更の場合
テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。
変更内容について何を確認したか、どういう方法で確認をしたかなど
のコマンドをして
editor-css/* inc/* lib/* options-css/* src/*以下のファイルが削除されることを確認
確認URL
( どこかのデモサイトかテストサーバーにデプロイ済みなどで確認できる場合はそのURL )
レビュワーに回す前の確認事項
レビュワー確認方法・確認内容など
・stableタグなどを利用して無料版へのアップデートを行う。
・inc内のファイルが削除されていることを確認
例)inc/template-tagsなど
・無料版のプラグインでエラーが出ないか確認
srcフォルダは今まで通りなのでブロックなどには変わりがないと思います。
・もしかするとSVNでファイルの削除を行う必要がある(かも?)
緊急では無いので慎重に行ってください。
レビュワー向け
レビュワーが確認して変更が反映されていない場合の確認事項
レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。