[github/laravel]OSSにプルリクを投げて初マージされました

mysql

laravel-short-urlというlaravelで短縮URLを作成できるcomposerライブラリがあるのですが、
6系で使ったところマイグレーションエラーが出ていて、内容も難しくなかったのでgithubで初プルリクエストを出してみました。

backlogはよく使ったりしているけど、githubでプルリクを出すことないので、ちょっととまどいました..😑
githubでのプルリクエストの手順と今回のエラー(mysql 1215 error)の対処方法を書いています。

プルリクエスト手順(pull request)

githubでforkする

対象のリポジトリのトップページに行って、右上のforkボタンを押します。

修正を行う

自分のリポジトリとして、forkされるので、git cloneして修正を行います。

// 今回は下記forkリポジトリ
git clone https://github.com/YasuakiHirano/laravel-short-url.git

ブランチは何を使っても大丈夫です。

pushして、プルリクを作る

修正が終わって、コミット&プッシュした後に、自分のリポジトリからnew pull requestを押して、プルリクエストを作成します。

コメントには修正内容などを載せたらいいと思います。
issueと紐付けたい時は「#」から始まる番号をコメントに書いておけばOKです。

今回はissueが20番で上がっていたので、
#20 bugfixで詳細にはgoogle翻訳で翻訳した英語を載せて送ってみました。

マージされた

翌日みたらマージされてました。
他国の人とやり取りするのはハワイ旅行ぶり。。(何年前・・・)

エラー内容(mysql 1215 error)

詳細

mysqlを使って、migrationした時に下記エラーが出ていました。
内容としてはusersテーブルに外部キーを貼ろうとしていたんですが、キーの型が違ったためエラーが発生していました。

Migrating: 2020_03_04_214540_add_user_id_to_urls

   Illuminate\Database\QueryException 

  SQLSTATE[HY000]: General error: 1215 Cannot add foreign key constraint (SQL: alter table `shorturl_urls` add constraint `shorturl_urls_user_id_foreign` foreign key (`user_id`) references `users` (`id`))

  at vendor/laravel/framework/src/Illuminate/Database/Connection.php:671

解決方法

こんな感じでintegerからunsignedBigInteger(usersのidと同じ型)に修正しました。

おわりに

結構簡単にプルリクを出せることもことがわかったので、
直せそうなissueを発見した時はどんどん修正して、送ってみたいと思います💪🏻

コメント

タイトルとURLをコピーしました