Skip to content

Commit a3eafef

Browse files
Merge pull request #1650 from Mark-Simulacrum/fix-errors
Fix a couple errors
2 parents aa68ceb + 6f81f42 commit a3eafef

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

src/github.rs

+2
Original file line numberDiff line numberDiff line change
@@ -888,6 +888,8 @@ pub enum IssuesAction {
888888
ReadyForReview,
889889
Synchronize,
890890
ConvertedToDraft,
891+
AutoMergeEnabled,
892+
AutoMergeDisabled,
891893
}
892894

893895
#[derive(Debug, serde::Deserialize)]

src/handlers/notification.rs

+8-1
Original file line numberDiff line numberDiff line change
@@ -151,7 +151,14 @@ async fn id_from_user(
151151
let team = match github::get_team(&ctx.github, team).await {
152152
Ok(Some(team)) => team,
153153
Ok(None) => {
154-
log::error!("team ping ({}) failed to resolve to a known team", login);
154+
// If the team is in rust-lang*, then this is probably an error (potentially user
155+
// error, but should be investigated). Otherwise it's probably not going to be in
156+
// the team repository so isn't actually an error.
157+
if login.starts_with("rust") {
158+
log::error!("team ping ({}) failed to resolve to a known team", login);
159+
} else {
160+
log::info!("team ping ({}) failed to resolve to a known team", login);
161+
}
155162
return Ok(None);
156163
}
157164
Err(err) => {

0 commit comments

Comments
 (0)