CSVのフォーマットを間違った場合の対処方法

[yasulab]

誤ってマージしてしまった場合、どうするか考えたい🤔

cc/ @NorifumiOgawa @miya0001

失敗したケース

• CoderDojo 嘉手納用のサーバーを申請します。 CoderDojo 嘉手納用のサーバーを申請します。 #16
• カンマ抜けの修正 カンマ抜けの修正 #17

対応案

• CSVフォーマットをテストして、失敗した場合はサーバーを生成しない?

[yasulab]

ワークアラウンドな対応

これで行けそうな気がするので、ちょっと試してみます 🔧 💨

1. コントロールパネルからサーバーをシャットダウン
2. コントロールパネルからサーバーを削除
3. CSVから当該業を削除
4. もう一回、PRを送り直してもらう

[yasulab]

@NorifumiOgawa ちょっと上の方法で作業をしてみました。もう一回PR送ってもらってもいいですかね?

[yasulab]

ワークアラウンドな方法で対応できるかな...? 🤔
CoderDojo 嘉手納用のサーバー申請です。 #19

[yasulab]

お、この方法でなんとかなりました! 🔧 💨

9421c48

今回は運用で対応していますが、理想的にはテストなどで対応した方が良いと思うので、この Issue は閉じずにこのままにしておきますね ;)

[miya0001]

昨日、CSVの列の数と簡単な正規表現のチェックをするようにTravis CIで仕込みました。
https://github.com/coderdojo-japan/dojopaas/blob/master/test/test-csv.js

それでもマージはできてしまうので、それを防ぐために以下のようにGitHubで以下のように設定してもらうと、問題が治らないとマージできなくなります。

ただ、この設定をするための権限がぼくになかったので @yasulab さんでやってもらっていいですか？

テストがコケたときのプルリクエストに対する修正はご本人でもコミッターでも可能です。

[yasulab]

設定しました! これで対応できたと思うので、この Issue は Close しちゃいますね ;)

[miya0001]

ダメなCSVフォーマットでプルリクが来ると以下のような感じになります。

これでもマージはできてしまうので、気持ち悪ければ Include administrators にチェックをいれるとそれもできなくなります。

[yasulab]

@miya0001 補足ありがとうございます! 😸

赤文字になって注意喚起はできていますし、仕様変更などに伴ってテストの方が間違ってしまうケースもありうる思うので、とりあえずはこのままで行ってみようかなと考えています ;)