昨今のJavaScriptをレビューしたら全くわからなかった話 - aoma blog

2019/09/13 633 users 昨今 aoma blog JavaScript class

はじめに 最近JavaScriptのプルリクが来たので「よーしやったるでー」って感じでレビューしたんですが、まったく理解できずレビューになりませんでした。 その時疑問に思ったことをメモします。。 誰かの助けになれば幸いです。。 これはもう引退やな。。 classの中で function って書くの省略してるの？ class TestClas... 続きを読む

初心者がプルリクまでに覚えるべきたった 9つの厳選 Gitコマンド - akiyoko blog

2014/12/03 508 users Git Qiita akiyoko blog 初心者 ネタ

2014-12-02 初心者がプルリクまでに覚えるべきたった 9つの厳選 Gitコマンド Git この投稿は 「Git Advent Calendar 2014 - Qiita」 の 2日目の記事です。 2年前の 「Git Advent Calendar 2012 - Qiita」 では、「Gitコマンド総選挙」と題して、本当に使える Git コマンドのベストテン発表というネタを書いたのですが、今... 続きを読む

今すぐ始められるOSS活動 - Gunosy Tech Blog

2020/08/13 479 users OSS活動 Gunosy Tech Blog Issue

はじめに Github Help Wantedでissueを選ぶ OSSのコントリビュート方法を確認する 実装 既存コードの動作確認 機能実装 プルリクを出す マージされる 最後に はじめに こんにちは。広告技術部のjohnmanjiroです。普段は広告配信のAPIや管理画面を作っています。 今回は最近自分が始めたOSS活動のやり方について紹介しよ... 続きを読む

Forkwell Press – 君がッ！ 泣いてマージするまで、プルリクを送ることをやめないッ！－トレジャーデータ 上薗竜太氏

2017/01/24 424 users Forkwell Press トレジャーデータ Rマッパー

この連載 では、「 Forkwell Jobs 」の開発にも関わるフリーランスエンジニアの後藤大輔 ( @idesaku ) が、さまざまな企業で働くエンジニアとリレー形式で対談を行っていきます。 前回のセオ商事・新多真琴氏からご紹介のあった第10回目のゲストは、トレジャーデータの上薗竜太 ( @kamipo ) 氏です。 上薗氏については、O/Rマッパー「Active Record」における「R... 続きを読む

エンジニアたちの面談してたんだけど、最近の子ほんと「一生エンジニア」やりたがるのね。30歳さしかかるんだからそろそろ部下を持てと言ってるんだけどなぁ

2024/03/26 398 users 部下 一生エンジニア エンジニアたち Issue レイヤー

あゆゆん♪ @ayu_littlewing 今度はエンジニアたちの面談してたんだけど、最近の子ほんと「一生エンジニア」やりたがるのね。 30歳さしかかるんだからそろそろ部下を持てと言ってるんだけどなぁ。 いつまでも振られたIssueをプルリクにするお仕事だけしてても今後のキャリアどうするんだ？？ エンジニアにもレイヤーはあ... 続きを読む

Forkwell Press – 「この境地にいるのは自分だけ」− 自身を追い込み続けてたどり着く「高み」とは − トレジャーデータ...

2017/01/30 303 users 境地 Forkwell Press トレジャーデータ 自身

この連載 では、「 Forkwell Jobs 」の開発にも関わるフリーランスエンジニアの後藤大輔 ( @idesaku ) が、さまざまな企業で働くエンジニアとリレー形式で対談を行っていきます。 今回のゲストは、トレジャーデータの @kamipo さんこと上薗竜太さんです。 インタビューは3部構成にてお届けしています。 第1部：君がッ！ 泣いてマージするまで、プルリクを送ることをやめないッ！ 第... 続きを読む

VTuberのホームページをGitHubのプルリクだけで更新していったらどうなるのか？企画に参戦してみた - Qiita

2020/01/19 277 users Twitter GitHub Qiita Vtuber 経緯

おめシスはいいぞ VTuberのホームページをGitHubのプルリクだけで更新していったらどうなるのか？という企画に参戦してみた。技術的にも非常に面白い試みだったので書き残しておく。 経緯 夜中にTwitterのタイムラインを眺めていたら興味深い動画を見かけた。 【検証】１ヶ月間、プルリクだけでホームページ作ったらどう... 続きを読む

GitHubが「Pull Panda」買収を発表。プルリクのSlackリマインダーやレビュアーの公平な割り当てなど、全機能が無料に － Publickey

2019/06/18 197 users GitHub Publickey レビュアー 買収 買収発表

GitHubは、GitHub関連ツールベンダ「Pull Panda」の買収を発表しました。この買収発表と同時に、Pull Pandaが提供していた3つのソフトウェア「Pull Reminders」「Pull Analytics」「Pull Assigner」はすべてGitHub Marketplaceで無料公開され、だれでも使えるようになりました。 Pull Remindersは、プルリクエストに対す... 続きを読む

GithubでPull Requestは絶対消せない。ヤバい＼(^o^)／ｵﾜﾀ - sinsokuのブログ

2013/10/15 193 users GitHub sinsoku pull request 結論

2013-10-16 GithubでPull Requestは絶対消せない。ヤバい＼(^o^)／ｵﾜﾀ git 先日、初めて知ったのですが、GithubのプルリクをCloseは出来ても、削除は出来ない。(*1) 普段使う上では困らないのですが、パスワードなどの公開すべきでない情報がプルリクに含まれると、結構ヤバい事になる。 結論 結論を先に書くと「リポジトリを作り直すしかない」 試したこと プルリ... 続きを読む

GitHubがコマンドラインツール「GitHub CLI」公開。コマンドラインからIssueやプルリクなど実行可能に。Windows、Mac、Linux対応 － Publickey

2020/02/13 168 users コマンドラインツール GitHub Publickey 実行

GitHubがコマンドラインツール「GitHub CLI」公開。コマンドラインからIssueやプルリクなど実行可能に。Windows、Mac、Linux対応 GitHubが新しいコマンドラインツール「GitHub CLI」をベータ版として公開しました。 Filter issues, create pull requests, checkout pull requests locally, and more—all from your comma... 続きを読む

コードレビューをしよう - pblog

2014/08/19 153 users Doorkeeper pblog ポエム ソニックガーデン

2014-08-19 コードレビューをしよう Sendagaya.rb #86 - Sendagaya.rb | Doorkeeperに参加してコードレビューについて思うところをみんなでディスカッションしたので、自分の思いをまとめておきます。（ポエム） ソニックガーデンの @mah_labさんのスライドにある7つの秘訣に同意です基本。あと、github上のプルリクでコードレビューするという前提でま... 続きを読む

3月1日、Homebrew のデフォルトの Python が Python 3 になります。 - methaneのブログ

2018/02/28 152 users Homebrew methane Python デフォルト

2018 - 02 - 28 3月1日、Homebrew のデフォルトの Python が Python 3 になります。 python 以前からアナウンスされていた 通り、 3/1 （日本時間では 3/2 になるかも）にデフォルトの Python が Python 3 に切り替わる予定です。 現在そのプルリク エス トがレビュー中です。 github.com 具体的には、今まで " python... 続きを読む

プルリクを起点に検証環境が自動で構築されるようにしたら すぐにレビューできるようになったのでみんなハッピーになれた話

2022/02/17 150 users 起点 検証環境 READY cohhei みんな

こんにちは。CacooチームのYAMLエンジニアの木村（@cohhei）です。「readiness」は「ready」の名詞形で「レディネス」と読むことをわりと最近知りました。今回はプルリクエスト向けに検証環境が構築される仕組みを作ったので紹介します。 忙しい人のためのざっくりとした説明 プルリクエストが作られたら自動で検証環境... 続きを読む

フリーランス歴20年の強強エンジニアからのガチコードレビュー集 - Qiita

2023/07/14 122 users Qiita 補助 Webエンジニア 職場 レビュー

はじめに こんにちは、まつけんです。 早いもので、Webエンジニアになって、10ヶ月経とうとしています。 先月末、僕の職場に参画していたフリーランス歴20年の強強エンジニアCさんが卒業されました。（以降Cさんと称します） Cさんには、いつも迅速かつ丁寧なレビューをしていただいてました。 たまに補助で僕のプルリク... 続きを読む

dockerをWebで管理できるshipyardが想像以上に便利だった件 - Qiita

2016/03/04 111 users Qiita Docker Container images

shipyardとは shipyard · shipyard dockerのcontainer、images等をWebで管理できちゃう ログインアカウントを作れる（読み取りのみ権限とかもOK） Registriesにも対応しているがまだv1のみの対応（プルリク来てたのでそのうち更新されるかも？） 検証環境（ホスト側） CentOS 7.2.1511 docker 1.10.2 インストール 続きを読む

Rails の issue を解決するまでの手順とOSS初心者でもできること - アジャイルSEの憂鬱

2019/10/16 104 users Rails Issue アジャイルSE バッチ コント

突然ですが、あなたはRailsのissueとプルリクがいくつあるかご存知でしょうか？ 2019年10月17日現在、それぞれ issue 384 / PR 803 になります。 多いですよね...。 個人的に、最近このissueを減らすのを少しでも手伝えないものかとissueにコメントしてみたり、パッチを書いたりしてるけど、 なかなか大変なので、コント... 続きを読む

LGTMでめでたさを伝えるChrome拡張をつくった - Thinking-megane

2013/12/24 104 users GitHub Thinking-megane LGTM ノリ

先日、高速にドッグフードを食べる方法を見て、LGTMとLGTM.inというサービスを知りました。 LGTMは、“looks good to me.”の略で、GitHubのプルリクに対するOKコメント、LGTM.inは、そのコメントにノリのいい画像を添えてspice upしようというサービスです。コードレビューの終わりにめでたさを伝えてもらえれば、字面だけのLGTMよりもうれしいし、そういう小さな承... 続きを読む

tigでコミットログ見てるときに「このコミットが含まれたプルリクに飛びたい」と思うときがあるので見れるようにした - パルカワ2

2017/02/09 94 users コミットログ tig commit ホームページビルダー

2017 - 02 - 09 tigでコミットログ見てるときに「このコミットが含まれたプルリクに飛びたい」と思うときがあるので見れるようにした @hisaichi5518 .tigrc とかで設定して、閲覧している commit をキーボードショートカットで開けるようにしておけば、commit のページに Pull Request へのリンクが含まれていそう — ホームページビルダー (@r7ka... 続きを読む

Rails の Issue とプルリクを毎日読むと勉強になる - アジャイルSEの憂鬱

2018/02/12 87 users Issue agile.esm Rails アジャイルSE

2018 - 02 - 12 Rails の Issue とプルリクを毎日読むと勉強になる rails 最近やっているけど、これ良い勉強になっているのでブログで紹介する。 読むようになったきっかけ 先月、永和システムマネジメントさんの OSS パッチ会に参加しました。 agile.esm.co.jp この会の懇親会では Rails の Issue やプルリクの話題が多く出ました。 ただ、私が知らな... 続きを読む

本の虫: GitHubで他人のプルリクエストに対しコンフリクト解消や追加の修正を行いつつマージする方法

2017/10/16 86 users GitHub レポジトリ コンフリクト コンフリクト解消 他人

読者がGitHubで何かを公開しているとしよう。そのレポジトリに対して他人がプルリクエストを送ってきた。なかなか良さそうな変更だ。早速マージしたい。 しかし、残念なことにそのプルリクをそのままマージすることができない。なぜならば、 コンフリクトを起こしている 追加の修正が必要だ こういう場合、大規模なプロジェクトや、PR主が職場の同僚や開発仲間であった場合、PR主に修正を依頼するものだ。しかし、個... 続きを読む

Yakst - より良いプルリクエストのための10のヒント

2015/01/19 85 users Yakst クローズドソース ヒント ソフトウェア チーム

良いプルリクエストを作ることには、良いコードを書くこと以上を含んでいる。 プルリクエストモデルは、チームでソフトウェアを開発するための素晴らしい方法になりつつある。チームメンバーが分散している場合は特にそうで、オープンソースの開発だけでなく、企業においてもそれは同じことだ。2010年頃から私は、オープンソースプロジェクトにおいてだけでなく、クローズドソースのソフトウェア開発のために内部的にプルリク... 続きを読む

Ruby学習者のためのリポジトリ「CodeYourRuby」をもっともっと広めたい - Qiita

2018/02/26 81 users Qiita リポジトリ コード お題 人向け

CodeYourRubyとは Ruby学習者が書いたコードを、Ruby上級者にレビューしてもらえるリポジトリ リポジトリにお題がコミットされているので、それを満たすコードを書いてプルリクを送るとレビューしてもらえる。 「一通りRuby本読んだけど、これで仕事できるレベルなのかよくわからないなあ。不安だなあ。」 という人向け リポジトリはコチラ code-your-ruby | GitHub Cod... 続きを読む

「おまえは今までレビューしたプルリクの数をおぼえているのか？」 - pixiv inside

2018/03/06 70 users pixiv inside おまえ

こんにちは、kanaです。社内ではpixivというサービスでPHPとTypeScriptとVim scriptを書く仕事をしています。今日はpixivの開発におけるコードレビューの話をします。 問題 pixivは昨年でサービス開始から 10周年 を迎えました。サービス開始当初と比較すると山のように新しい機能や画面が増えています。なのでpixivのコードベースは巨大です。PHPファイルだけでも500... 続きを読む

JavaScript - 自動でnpmライブラリをアプデ&プルリクしてくれるサービスのgreenkeeperがすごく便利だった - Qiita

2015/12/20 57 users Qiita npmライブラリ フル npm xmas みなさん

はじめに どうもみなさん npm xmas ！（気が早い） みなさんnpmライブラリのバージョンのアップデートどうしてますか？ 方法としては npm-check-updates などが有名なのではないかと思うのですが、逐一手動でアップデートするのは面倒ですよね？ そこで、自動でアップデートしてプルリク投げてくれるという greenkeeper というのを使ってみたら非常に便利だったので紹介したいと... 続きを読む

Androidでaptのライブラリを作るときの高速道路 - Qiita

2015/07/07 56 users Qiita apt 知見 ライブラリ Android界隈

はじめに Android界隈で最近aptを使ったライブラリが増えてきていると感じています。しかしapt自身、特にAndroidでの書き方を解説している日本語の記事は多くありません。ライブラリを書かないとしても、どのように動くかを知っておくことでエラーに対応しやすくなったり、プルリクを送りやすくなったり良いことがあると思うので、私の経験を踏まえてaptのライブラリを作るときの知見を共有したいと思いま... 続きを読む