がないだけでオール 100点にならなかったので出しました。
せっかく[SEO-friendly -Gridsome]を特徴として謳っているので 100点にしておきたいところです。これは設定ファイルにデフォルト値をセットしておくだけで 100点になります。
なにより設定にデフォルトが入っていないと、設定箇所がないように見えるので開発者が気づかない、または調べるのを後回しにして設定忘れが放置される可能性もありえます。デフォルト値は意味のない値とはいえ設定が明示的に入っていることで開発者に意識を向けさせることができるのではないでしょうか。
本質的ではないですが、気になったのでプルリクを出すことにしました。
関連コンテンツ
こちらのプルリクは、Hexoのデフォルトテーマlandscapeの Twitter 共有リンクを開いたときに URL のみだったのを、タイトルも含めるようにしたものです。
カック@ブロガー / k9u(@kakakakakku)さんに指摘されるまで気づきませんでした。。。自分が使っているものをしっかり把握しておかないとですね 😓
この共有リンクはテーマのデフォルトではダイアログに入っているのですが、本ブログでは展開して記事の一部に表示する改造をしています。そのため単純な差分が取れずに、よくコードを確認する必要がありましたが勉強になりました。また最新のコードを本ブログのテーマに取り込むきっかけにもなりよかったです。(さりげなく本家からマージした LinkedIn の共有ボタンが入ってる)
テーマ周りでも Issues が上がっていたりするので、引き続き貢献ポイントを探して活動していきたいです。
関連コンテンツ
こちらのプルリクは、本ブログの開発環境を Node.js v8 から v10 にバージョンアップしたところエラーが発生したので修正したものになります。
このsfarthin/ga-analyticsは、Google Analytics からレポートデータを取得するもので、本ブログでは Hexo で人気記事のリストを実現するtea3/hexo-related-popular-postsPlugin が使っています。エラーが発生しているので修正が必然だったものになります。
関連コンテンツ
5月は3件のプルリクを出せました。
まだ、どれもマージされてないのが寂しいですが、最新の2件はプロジェクトテンプレートやテーマですし内容も優先度が高くないので、じっくり待ち。
3つ目の sfarthin/ga-analytics #2 は、エラー修正なのでマージされると嬉しいな。最終更新が 2014年と5年前なのがちょっと心配。。。
気づいたところを少し修正しただけですが、結構楽しいです。
今後とも OSS へ貢献する活動をしていきたいと思います。
🚲 Let’s enjoyOSS-Friday!!
OSS-Friday 参考情報