CSVのフォーマットを間違った場合の対処方法 about dojopaas HOT 8 CLOSED

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

from dojopaas.

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

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

from dojopaas.

ワークアラウンドな対応

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

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

from dojopaas.

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

from dojopaas.

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

from dojopaas.

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

9421c48

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

from dojopaas.

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

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

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

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

from dojopaas.

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

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

from dojopaas.