forked from the-distro/ofborg
merge conflict labels: move
This commit is contained in:
parent
ed0882a975
commit
6cbaa5d5c6
|
@ -352,9 +352,21 @@ impl<'a> EvaluationStrategy for NixpkgsStrategy<'a> {
|
||||||
Ok(())
|
Ok(())
|
||||||
}
|
}
|
||||||
|
|
||||||
fn merge_conflict(&mut self) {}
|
fn merge_conflict(&mut self) {
|
||||||
|
update_labels(
|
||||||
|
&self.issue_ref,
|
||||||
|
&["2.status: merge conflict".to_owned()],
|
||||||
|
&[],
|
||||||
|
);
|
||||||
|
}
|
||||||
|
|
||||||
fn after_merge(&mut self, status: &mut CommitStatus) -> StepResult<()> {
|
fn after_merge(&mut self, status: &mut CommitStatus) -> StepResult<()> {
|
||||||
|
update_labels(
|
||||||
|
&self.issue_ref,
|
||||||
|
&[],
|
||||||
|
&["2.status: merge conflict".to_owned()],
|
||||||
|
);
|
||||||
|
|
||||||
status.set_with_description("Checking new stdenvs", hubcaps::statuses::State::Pending);
|
status.set_with_description("Checking new stdenvs", hubcaps::statuses::State::Pending);
|
||||||
self.check_stdenvs_after();
|
self.check_stdenvs_after();
|
||||||
|
|
||||||
|
|
|
@ -250,11 +250,9 @@ impl<E: stats::SysEvents + 'static> worker::SimpleWorker for EvaluationWorker<E>
|
||||||
|
|
||||||
info!("Failed to merge {}", job.pr.head_sha);
|
info!("Failed to merge {}", job.pr.head_sha);
|
||||||
|
|
||||||
update_labels(&issue_ref, &["2.status: merge conflict".to_owned()], &[]);
|
evaluaton_strategy.merge_conflict();
|
||||||
|
|
||||||
return self.actions().skip(&job);
|
return self.actions().skip(&job);
|
||||||
} else {
|
|
||||||
update_labels(&issue_ref, &[], &["2.status: merge conflict".to_owned()]);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if self
|
if self
|
||||||
|
|
Loading…
Reference in a new issue