Conversation

Notices

1. Embed this notice
   naskya::dev (dev@post.naskya.net)'s status on Wednesday, 26-Feb-2025 23:38:44 JST naskya::dev

   • SyoBoN

   @syobon@post.syobon.net Plegosa 0.4 のプルリクエストで私が勝手にコメントをつけた😇ところは別のプルリクエストを作ってほしいですか、それともこのままでいいですか
   https://github.com/syobocat/plegosa/pull/33

   • Embed this notice
     SyoBoN (syobon@post.syobon.net)'s status on Thursday, 27-Feb-2025 00:29:58 JST SyoBoN

     in reply to
     @dev こちらからコメントが見えてないかもです
   • Embed this notice
     naskya::dev (dev@post.naskya.net)'s status on Thursday, 27-Feb-2025 00:29:58 JST naskya::dev

     in reply to

     • SyoBoN

     @syobon@post.syobon.net そんなことある！？

   • Embed this notice
     naskya::dev (dev@post.naskya.net)'s status on Thursday, 27-Feb-2025 00:31:54 JST naskya::dev

     in reply to

     • SyoBoN

     @syobon@post.syobon.net ん、画像が圧縮されてしまって読めないかも
     https://i.imgur.com/7dV2jjj.png

   • Embed this notice
     SyoBoN (syobon@post.syobon.net)'s status on Thursday, 27-Feb-2025 00:33:46 JST SyoBoN

     in reply to
     @dev こうなってますね…
     ありがとうございます、あとで確認しておきます
     naskya::dev likes this.
   • Embed this notice
     naskya::dev (dev@post.naskya.net)'s status on Thursday, 27-Feb-2025 00:38:24 JST naskya::dev

     in reply to

     • SyoBoN

     @syobon@post.syobon.net GitHub にもバグがあると思うとちょっと穏やかな気持ちになる

   • Embed this notice
     SyoBoN (syobon@post.syobon.net)'s status on Thursday, 27-Feb-2025 00:49:12 JST SyoBoN

     in reply to

     • SyoBoN

     @dev Eqのderiveに関してはClippy(のnursery)に「Eqもついでにderiveできるよ」と言われたのでつけました (毎回nurseryとpedanticは一通り目を通していて、消せるものは消しておいた方が見やすかったので)

     他に関しては指摘の通りですね、ありがとうございます

     warning: you are deriving `PartialEq` and can implement `Eq` --> src/config.rs:12:23 | 12 | #[derive(Deserialize, PartialEq, Debug)] | ^^^^^^^^^ help: consider deriving `Eq` as well: `PartialEq, Eq` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#derive_partial_eq_without_eq = note: `-W clippy::derive-partial-eq-without-eq` implied by `-W clippy::nursery` = help: to override `-W clippy::nursery` add `#[allow(clippy::derive_partial_eq_without_eq)]`
   • Embed this notice
     naskya::dev (dev@post.naskya.net)'s status on Thursday, 27-Feb-2025 00:49:12 JST naskya::dev

     in reply to

     • SyoBoN

     @syobon@post.syobon.net なるほど（普通は derive しておいても良さそうな Debug とかまで使わないところでは削られていたのでコンパイル時間を詰めたくてそうしているのかと解釈してしまった

   • Embed this notice
     naskya::dev (dev@post.naskya.net)'s status on Thursday, 27-Feb-2025 00:58:54 JST naskya::dev

     in reply to

     • おだら
     • SyoBoN

     @s3_odara@mastodon.hakurei.win @syobon@post.syobon.net Leave comment はしたんですけど、もしかしたらその後に更に Submit review が必要なのかもしれない（今そのボタンを発見した）
     よく分からない……

   • Embed this notice
     おだら (s3_odara@mastodon.hakurei.win)'s status on Thursday, 27-Feb-2025 00:58:55 JST おだら

     in reply to

     • SyoBoN

     @dev @syobon Githubのレビュー、コメントを付けたあとにRequire changesかleave commentかAcceptかのアクションを起こさないと反映されなかったと思います。
     この場合だとLeave commentみたいなボタンでいいはず。

   • Embed this notice
     おだら (s3_odara@mastodon.hakurei.win)'s status on Thursday, 27-Feb-2025 01:14:20 JST おだら

     in reply to

     • SyoBoN

     @dev @syobon あ、出てきました

     naskya::dev likes this.