Skip to content

Commit c390cd1

Browse files
authored
Remove 2.status: work-in-progress label support (#707)
This label is being removed, as discussed in NixOS/nixpkgs#419944
1 parent 4caed13 commit c390cd1

File tree

2 files changed

+3
-25
lines changed

2 files changed

+3
-25
lines changed

README.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,8 +26,8 @@ When opening a PR with multiple commits, ofborg creates a single build job for
2626
all detected packages. If multiple commits get pushed to a PR one-by-one, each
2727
detected package will get a separate build job.
2828

29-
If the title of a PR begins with `WIP:`, contains `[WIP]` anywhere, or has the
30-
`2.status: work-in-progress` label, its packages are not built automatically.
29+
If the title of a PR begins with `WIP:` or contains `[WIP]` anywhere, its
30+
packages are not built automatically.
3131
**Note**: Marking a PR as a draft does not prevent automatic builds.
3232

3333
## Commands

ofborg/src/tasks/evaluate.rs

Lines changed: 1 addition & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -483,29 +483,7 @@ pub fn update_labels(issueref: &hubcaps::issues::IssueRef, add: &[String], remov
483483
}
484484

485485
fn issue_is_wip(issue: &hubcaps::issues::Issue) -> bool {
486-
if issue.title.contains("[WIP]") {
487-
return true;
488-
}
489-
490-
if issue.title.starts_with("WIP:") {
491-
return true;
492-
}
493-
494-
issue.labels.iter().any(|label| indicates_wip(&label.name))
495-
}
496-
497-
fn indicates_wip(text: &str) -> bool {
498-
let text = text.to_lowercase();
499-
500-
if text.contains("work in progress") {
501-
return true;
502-
}
503-
504-
if text.contains("work-in-progress") {
505-
return true;
506-
}
507-
508-
false
486+
return issue.title.starts_with("WIP:") || issue.title.contains("[WIP]");
509487
}
510488

511489
/// Determine whether or not to use the "old" status prefix, `grahamcofborg`, or

0 commit comments

Comments
 (0)