danderson/ofborg
0
0
Fork 0
forked from the-distro/ofborg
Code Pull requests Activity

eval/nixpkgs: add note about why the extra curly braces are necessary

Browse source
This commit is contained in:
Cole Helbling 2023-09-15 06:42:25 -07:00
parent e043e3d9dd
commit de415d3729
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
ofborg/src/tasks/eval/nixpkgs.rs
View file

@ -16,7 +16,6 @@ use crate::tasks::evaluate::{get_prefix, make_gist, update_labels};
use std::path::Path; use std::path::Path;
use brace_expand::brace_expand;
use chrono::Utc; use chrono::Utc;
use hubcaps::checks::{CheckRunOptions, CheckRunState, Conclusion, Output}; use hubcaps::checks::{CheckRunOptions, CheckRunState, Conclusion, Output};
use hubcaps::gists::Gists; use hubcaps::gists::Gists;
@ -626,7 +625,10 @@ fn parse_commit_messages(messages: &[String]) -> Vec<String> {
// Convert "foo: some notes" in to "foo" // Convert "foo: some notes" in to "foo"
line.split_once(':').map(|(pre, _)| pre.trim()) line.split_once(':').map(|(pre, _)| pre.trim())
}) })
.flat_map(|line| brace_expand(&format!("{{{}}}", line))) // NOTE: This transforms `{foo,bar}` into `{{foo,bar}}` and `foo,bar` into `{foo,bar}`,
// which allows both the old style (`foo,bar`) and the new style (`{foo,bar}`) to expand to
// `foo` and `bar`.
.flat_map(|line| brace_expand::brace_expand(&format!("{{{}}}", line)))
.map(|line| line.trim().to_owned()) .map(|line| line.trim().to_owned())
.collect() .collect()
} }