✨ alert trait

[comavius]

関連Issue

• close Alert traitを作る #87

概要

alertを出すときに使うtrait

変更内容

• Alert traitを追加

チェックリスト

• テストが通っている
• 下記のいずれかを満たしている
  • • 変更点についてテストを追加/修正した
  • • テストを追加するissueを立てた
  • • テストが不要な変更である
• レビュワーを指定した
• タグをつけた

補足

[kenken714]

async traitにしてください

[kenken714]

send_alert の引数で、Option に kind があると嬉しい

[kenken714]

send_alert の引数で、Option に kind があると嬉しい

うそ

[kenken714]

enumに対して Display を実装すればよい
ex:

enum AlertKind {
    Error(String),
    Note(String),
}

impl fmt::Display for AlertKind {
    fn fmt(&self, f: &mut fmt::Formatter) -> fmt::Result {
        match *self {
            AlertKind::Error(ref s) => write!(f, "Error: {}", s),
            AlertKind::Note(ref s) => write!(f, "Note: {}", s),
        }
    }
}

pub trait Alert {
    fn send_alert<T: std::fmt::Display>(&self, message: T);
}

impl Alert for AlertKind {
    fn send_alert<T: std::fmt::Display>(&self, message: T) {
        println!("{}", message);
    }
}

[kenken714]

LGRM

[kenken714]

lgtm2