Closed ehuss closed 2 months ago
The validate_config handler seems to trigger on events hitting old, closed PRs. It loads the triagebot.toml from the PR itself, which might be very old, and fail. https://github.com/rust-lang/rust/pull/73206#issuecomment-2089379975 is an example.
validate_config
triagebot.toml
I think at a minimum it probably should not run on closed PRs. I'm not sure if it should ignore other events, but that seems like a start.
The
validate_confighandler seems to trigger on events hitting old, closed PRs. It loads thetriagebot.tomlfrom the PR itself, which might be very old, and fail. https://github.com/rust-lang/rust/pull/73206#issuecomment-2089379975 is an example.I think at a minimum it probably should not run on closed PRs. I'm not sure if it should ignore other events, but that seems like a start.