Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

東京Ruby会議12のレポート chihaso 担当分 #565

Open
wants to merge 5 commits into
base: 0065-tokyo12
Choose a base branch
from

Conversation

chihaso
Copy link

@chihaso chihaso commented Feb 7, 2025

  • morihirok『混沌とした例外処理とエラー監視に秩序をもたらす』
  • アンチボッチランチ

記載場所など含め、ご確認お願いいたします!🙇‍♂️

- morihirok『混沌とした例外処理とエラー監視に秩序をもたらす』
- アンチボッチランチ
Copy link

@pndcat pndcat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

レポートありがとうございます!
いくつかコメントを入れたのでご確認お願いします。


そこで、RuboCopにより、すべてのエラーを丸ごとハンドリングしてしまうコードや、Un Expectedなエラーをハンドリングしてしまっているコードを取り締まるようにしました。これにより、同様の問題を抱えるコードの増加を防ぐことができ、既存の問題箇所は .rubocop_todo.yml にまとめられました。また、RuboCop違反によって実装時に立ち止まって考えるタイミングを得られるようになったとのことです。

さらに、Expectedなエラーであってもログに残すべきケースでは、エラートラッカーは使用せず、ログの用途に応じて適切な送り先を選択します。例えば、一定頻度以上で発生するエラーの検知にはDatadogやNewRelicなどのAPMを使用し、エラー発生後に処理が必要な場合にはデータベースや検索性の高いストレージに記録しているとのことです。
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

半角スペースをお願いします!

Suggested change
さらに、Expectedなエラーであってもログに残すべきケースでは、エラートラッカーは使用せず、ログの用途に応じて適切な送り先を選択します。例えば、一定頻度以上で発生するエラーの検知にはDatadogやNewRelicなどのAPMを使用し、エラー発生後に処理が必要な場合にはデータベースや検索性の高いストレージに記録しているとのことです。
さらに、Expectedなエラーであってもログに残すべきケースでは、エラートラッカーは使用せず、ログの用途に応じて適切な送り先を選択します。例えば、一定頻度以上で発生するエラーの検知にはDatadogやNew RelicなどのAPMを使用し、エラー発生後に処理が必要な場合にはデータベースや検索性の高いストレージに記録しているとのことです。

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestコミットさせていただきました!🙏
4317cee

筆者も初めてお会いする方々とランチをご一緒させていただきました。
道中一緒にプリクラを撮ったり、一人減ったと思ったらなぜか三人増えたり、お店がなかなか見つからなかったりといろんな事がありつつ、発表内容や普段の開発についてたくさんお話できて、とても楽しい時間を過ごす事ができました。
筆者は技術イベントが好きでよく参加はするものの、知らない人の中に入っていくのはなかなか難しく、他の参加者の方々との交流はあまりできていませんでした。
今回の企画は、筆者のような参加者には非常にありがたく、まさに地域.rbらしい内容だったと感じました。
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

まさに地域.rbらしい内容だった

もしかして「地域Ruby会議」のことですか?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!
「地域.rbから発展したイベント」みたいな意図だったのですが、地域Ruby会議の方が適切でした。
修正しました🙏
d708f11

道中一緒にプリクラを撮ったり、一人減ったと思ったらなぜか三人増えたり、お店がなかなか見つからなかったりといろんな事がありつつ、発表内容や普段の開発についてたくさんお話できて、とても楽しい時間を過ごす事ができました。
筆者は技術イベントが好きでよく参加はするものの、知らない人の中に入っていくのはなかなか難しく、他の参加者の方々との交流はあまりできていませんでした。
今回の企画は、筆者のような参加者には非常にありがたく、まさに地域.rbらしい内容だったと感じました。
ご一緒させていただいた皆様、ありがとうございました!
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

アンチボッチランチの一番下にもお名前をお願いします!

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

すみません忘れておりました・・・!追加しました!
7715d0c

@chihaso chihaso requested a review from pndcat February 11, 2025 23:58
@chihaso
Copy link
Author

chihaso commented Feb 22, 2025

アンチボッチランチの記事に改行を追加いたしました!34ddfe7
下図のような感じになっております。
スクリーンショット 2025-02-22 22 50 10

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants