「履歴書にはGitHubアカウントを書かない方が良い説」記事へのコメント

そうだ、hugに置き換えよう

[Anonymous Coward]

「ソースコードの内容が酷い」

「ソースコードが酷い」ではなく「内容が酷い」というのはなんだろう？
変数名にfuckやsuckが多用されてるとか？

Re:

[Anonymous Coward]

文字通り"内容"がひどいんじゃない？

酷いソースコードでも、
インデントされていない、適切なコメント管理ができていない
など見た目の時点でNGなコードもあるんだろうけど、

一見綺麗に書かれた様に見えるが、
明らかなバグがある、同様のコードが複数個所に存在する、未使用のコードが多々存在する
など中身が酷いコードもあるってことじゃない？

hylomがそこまで考えて文章書いてるのか知らんけどね

Re:

[Anonymous Coward]

Dry原則を絶対視している時点で真っ当じゃない
明らかなバグと言うがバグ修正中かもしれない
未使用のコードも開発中かもしれない

色々とDry原則を絶対視していたりする時点でお前が何を行ってんだ

Re:

[Anonymous Coward]

そういう細かいところに突っかかるなって
Dry原則を絶対視なんてしてないだろうが、
いわゆるコードスメル的に 酷い内容に見えるコード があるんじゃないか？
って言ってるだけだろ？

それとも何？コードスメル も知らないの？

後さ、お前こそ、開発中の動かないコードを公開リポジトリにpushするなよ
何のための分散リポジトリだよ。チーム開発したことないの？

まぁ、元記事で言われてる様な酷い内容のリポジトリを公開している人が、スラドにもいるってことは分かったよ

Re:

[Anonymous Coward]

いや、個人のGithubアカウントと個人のレポジトリの話でしょ？
動かないコードpushしておいたって別なPCで作業継続するためとか作業履歴として別に問題ないよ？
お前作業ブランチを切り分けることすらしないの？
別に雑にコード書いといて後からリファクタリングでなおす事もあるわけで
動かないコードを自分のレポ(ブランチ)にpushすることの何が問題なの？
別にマージされてるわけでもなく各個人がブランチ切ってるんだし
プルリクしてマージされなきゃ影響ないわけでお前Git使ったことないんか？

お前の言ってることこそ「何のための分散レポジトリだよ」案件なんだけど。
もしかしてこれらの作業を全て終わらせたものを一度マージしてgithubにpushしろって言ってる？
そんなもんプルリク送る前にやりゃいいだろ

Re:

[nnnhhh (47970)]

githubを清書置き場、リリース物置場としか見ない人が面接官に多いってのはありそうですね
そういう人が「ここはローカルリポジトリ」なタイプの人のをみると文句言いそう

Re:

[Anonymous Coward]

あーそういう人は居ますね
インターネットは雑多な場所なのに綺麗なものだけおけって主張は分からん

SVNで作業途中なのに入れちゃったならわかるけどgitならマージなりしなきゃ影響受けないのに
なんで作業経過を入れんなになるのか分からん
出来てなくってもとりあえずコミットしておくしプルリクするときだけまとめてから送るけど普通じゃないのか、これ。
俺が知る限りGAFAのエンジニアでもそうだった人は居たけど
基本的に作業経過を残す事でバグとかどこで入れちゃったのか追っかけられるし。

Re:

[Anonymous Coward]

イキって履歴書に書くくらいのリポジトリなら、
人に見せれる清書置き場やリリース物置き場 として見るのが普通だよね？
前提間違ってない？

作業履歴をこまめに残しておくことは自分も賛成だけど、
そんなローカル扱いのゴミ箱リポジトリを、一般公開するだけならまだしも、履歴書に書くなって話だよ。

Re:そうだ、hugに置き換えよう

[Anonymous Coward]

どーかな
履歴書は手書きで書こう見たいな古い価値観な気がしないでもないな