issue #127 READMEに参加方法を追記 ＆ MEMBERS.md の叩き台設置

[KEINOS]

対象トピック番号

• issue #127 「サークルメンバーをREADMEに記述するのはどうでしょう？」 by @alice1017

補足

• README.md の「サークル参加方法」追加
• MEMBERS.md の作成

TL;DR（進捗・結論 2018/09/10 現在）

• 審議中 ( ´・ω) (´・ω・) (・ω・｀) (ω・｀ )

[alice1017]

SiderのESlintでMarkdownがエラーになっちゃうの、どうにかできないんですかねぇ

[KEINOS]

なんぞ？他の Markdown ファイルは通るのに。

[alice1017]

ログ見ると他のmdファイルもエラーでてますね

/tmp/d20180910-7-1p4xjrx/CODE_OF_CONDUCT.md: line 1, col 1, Error - Parsing error: Unexpected character '#'
/tmp/d20180910-7-1p4xjrx/composer.json: line 2, col 11, Error - Parsing error: Unexpected token :
/tmp/d20180910-7-1p4xjrx/index.php: line 1, col 1, Error - Parsing error: Unexpected token <
/tmp/d20180910-7-1p4xjrx/LICENSE_OUTLINE.md: line 1, col 1, Error - Parsing error: Unexpected character '#'
/tmp/d20180910-7-1p4xjrx/LICENSE.md: line 1, col 24, Error - Parsing error: Unexpected token 4.0
/tmp/d20180910-7-1p4xjrx/MEMBERS.md: line 1, col 1, Error - Parsing error: Unexpected character '#'
/tmp/d20180910-7-1p4xjrx/README.md: line 1, col 10, Error - Parsing error: Unexpected token Status
/tmp/d20180910-7-1p4xjrx/sideci.yml: line 4, col 10, Error - Parsing error: Unexpected token .

[KEINOS]

むふぅ。Sider がゲップしまくってる。何が原因？

2018-09-10T07:13:12ZRunning analyzer
2018-09-10T07:13:12Z
# eslint --format=compact --no-color --output-file=/tmp/d20180910-7-1p4xjrx/output.txt ./*
2018-09-10T07:13:13Zexit status: 1
2018-09-10T07:13:13Z
output.txt
2018-09-10T07:13:13Z
/tmp/d20180910-7-1p4xjrx/CODE_OF_CONDUCT.md: line 1, col 1, Error - Parsing error: Unexpected character '#'
/tmp/d20180910-7-1p4xjrx/composer.json: line 2, col 11, Error - Parsing error: Unexpected token :
/tmp/d20180910-7-1p4xjrx/index.php: line 1, col 1, Error - Parsing error: Unexpected token <
/tmp/d20180910-7-1p4xjrx/LICENSE_OUTLINE.md: line 1, col 1, Error - Parsing error: Unexpected character '#'
/tmp/d20180910-7-1p4xjrx/LICENSE.md: line 1, col 24, Error - Parsing error: Unexpected token 4.0
/tmp/d20180910-7-1p4xjrx/MEMBERS.md: line 1, col 1, Error - Parsing error: Unexpected character '#'
/tmp/d20180910-7-1p4xjrx/README.md: line 1, col 10, Error - Parsing error: Unexpected token Status
/tmp/d20180910-7-1p4xjrx/sideci.yml: line 4, col 10, Error - Parsing error: Unexpected token .

8 problems
2018-09-10T07:13:13Z
# eslint --version
2018-09-10T07:13:14Z
v5.4.0
2018-09-10T07:13:14Zexit status: 0
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected character '#'
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected token :
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected token <
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected character '#'
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected token 4.0
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected character '#'
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected token Status
2018-09-10T07:13:14Z
No id found! - Error - Parsing error: Unexpected token .
2018-09-10T07:13:14Z
Removing issues from unchanged or untracked files...
2018-09-10T07:13:14Z
    -> 6.7431e-05s
2018-09-10T07:13:14Z
Writing result...
2018-09-10T07:13:14Z
    -> 0.000237034s

[KEINOS]

ログには色々出てるけど、管理画面では MEMBERS.md だけが対象なのよね。 どっかで除外設定た記憶もナッシング。

[alice1017]

そもそもeslintでmarkdownをチェックする必要あるんですか？ sideci.ymlでディレクトリの除外とかできないんですかね

[KEINOS]

この前も ESLint が v5系になってからエラーが出るようになったので、問い合わせたら仕様と言われたので、ESLint から "*.js" 以外を除外にした方がいいのかも。

TL;DR

ESLint を Sider 側のリポジトリ設定で有効にした状態で “sideci.yml” の “linter:” に “eslint” がない場合、デフォルトで使われる dir 設定が “.” であるため、レビューに失敗する。

linter:
  eslint:
    dir: “./*”

と設定すると回避できる。デフォルトの ESLint のバージョンが v4.x 系から v5.1.0 になったことによるとおもわれる。

ご連絡いただきありがとうございます。 お問い合わせいただいた件について調査したところ、これは、解析対象となるディレクトリ内に、 JavaScriptファイルが存在しないために発生するメッセージであり、ESLintのデザイン通りの挙動です。

v4系で発生しなかったのは、4系では解析対象のファイルが存在しなかったときは Success で返す仕様になっていたためです。

したがって、お客様がJSファイルを追加した場合は、この No files matching the pattern "." were found. というメッセージはなくなるかと思います。

[KEINOS]

Sider ESLint マニュアル https://help.sider.review/tools/javascript/eslint#options を見て以下を設定。NG。

  eslint:
    dir: "./*"
    options:
      ext: "js"

ってか、Sider のログみて気づいたけど CI の設定の入れ子がおかしくないか？

CI Configuration
  {:linter=>
    {:code_sniffer=>
      {:version=>3,
       :dir=>"./",
       :options=>
        {:extensions=>"php,inc", :standard=>"PSR2", :encoding=>"utf-8"}},
     :phpmd=>
      {:options=>{:suffixes=>"php,inc", :exclude=>"vendor/", :strict=>true}},
     :eslint=>{:dir=>"./*", :options=>{:ext=>"js"}}}}

[KEINOS]

そもそもeslintでmarkdownをチェックする必要あるんですか？ @alice1017

ね。デフォルトで Javascript になっているはずなのに。'js'ファイルに限定もできない。

[KEINOS]

CI config の JSON、整形したらちゃんとなってた。。。

{
  :linter=>
  {
    :code_sniffer=>
    {
      :version=>3,
      :dir=>"./",
      :options=>
      {
        :extensions=>"php,inc", :standard=>"PSR2", :encoding=>"utf-8"
      }
    },
    :phpmd=>
    {
      :options=>
      {
        :suffixes=>"php,inc", :exclude=>"vendor/", :strict=>true
      }
    },
    :eslint=>
    {
      :dir=>"./*",
      :options=>
      {
        :ext=>"js"
      }
    }
  }
}

[alice1017]

とりあえずなんとかなりましたね！

[KEINOS]

@alice1017

いま怒りの矛先を Qiita 記事にして記事書いてるw

[alice1017]

nodejs取り扱うまで、SiderのESLintを使わないっていうのは出来ないんですか？

[KEINOS]

nodejs取り扱うまで、SiderのESLintを使わないっていうのは出来ないんですか？ @alice1017

そうですね。dummy.js もなんか気持ちわるいですし。 issue あげてみ 💪

[KEINOS]

@alice1017 Issue あげてきました #132

[hidao80]

@alice1017 不具合発見、ありがとうございます！

@KEINOS 対応ありがとうございます！

132 が解決し次第、コミットお願いします！

[KEINOS]

@hidao80 （ @alice1017 ）

132 が解決し次第、コミットお願いします！

132 解決（Close）しマスター。この PR に各々プロフィールをコミットしていきます？それとも、一旦マージして、各々 PR ？

[hidao80]

@KEINOS

MEMBERS.mdのREADME へのリンク先URLが「URL」のままではありませんか？

そこだけ直してコミットしていただけるとありがたいです。🙏

[hidao80]

@KEINOS

一旦コミットして、各々 PR で良いと思います。 MEMBERS.mdの公開が急務だと考えます。

[alice1017]

@KEINOS

a48bfddce09a0e6c8359bd34762aec3db6d57d2b...7b72bf67b29e22ddb3bb6a81dd4b2c3ee4d409ea これらのコミットを rebase で一つにできませんか？ そのほうがログがキレイになると思います 😃

[KEINOS]

@alice1017

コミットをまとめたことないのですが squash と rebase の違いがよくわからない、、、

[alice1017]

こちらを参考にしてみてはどうでしょう

【git rebase -i】gitのcommitをまとめる - Qiita - https://qiita.com/tsuuuuu_san/items/f708a9f7ea8ab8eb6945

[KEINOS]

@alice1017

うひー、こえー。やってみる！

[alice1017]

お疲れ様です！

[KEINOS]

うまくいったの、、、か？

[KEINOS]

あっれぇ？失敗してる、、、

[KEINOS]

1. rebase したい１つ前に（ 6a92ab6 で）入って、

$ git rebase -i 6a92ab6

2. vim から a48bfdd ...7b72bf6 に s フラグに書き換えて
3. 保存したらコミットメッセージ入れろというのでコミットメッセージを入れて保存（ 9fd0d2c の内容）
4. $ git push -f origin master したのち
5. GitHub 上で PR をマージ
6. 作業ブランチを Master に変えて Pull したら上のような状態

なにが沖田艦長