rust-lang / highfive

Github hooks to provide an encouraging atmosphere for new contributors
Apache License 2.0
185 stars 128 forks source link

Avoid an assertion error on double-adding a group #424

Closed Mark-Simulacrum closed 2 years ago

Mark-Simulacrum commented 2 years ago

max_paths can contain the same group under multiple paths, I think, though I didn't follow the code super closely. In any case, it seems fine to just skip a double-added group rather than asserting it can't happen.