on master help sign in
commenter outdated: Also check test/benchmark versions
commitb81f8bd040152dbe550172ff023e5469dfcdd922 authorAdam Bergmark <adam@bergmark.nl> dateFri 7 Jan 2022, 17:01 (5 years ago) parent22ba2b0
2 files changed+22−20
build-constraints.yaml+5−7 etc/commenter/src/lib.rs+17−13
build-constraints.yaml+5−7
@@ -5204,18 +5204,18 @@ packages:
52045204 - boxes
52055205 - cassava
52065206 - charsetdetect-ae # #6326/closed
5207− - coercible-utils # 6271
5207+ - coercible-utils # #6271
52085208 - compiler-warnings # #6326/closed
52095209 - curl
52105210 - docopt # #6326/closed
52115211 - dynamic-state # #6326/closed
52125212 - first-class-patterns # #5965/closed
52135213 - ghc-trace-events # #6326/closed
5214− - hashing # 6271
5215− - hnix-store-core # 6271
5214+ - hashing # #6271
5215+ - hnix-store-core # #6271
52165216 - ilist # #5965/closed
52175217 - io-storage # #6326/closed
5218− - monadlist # 6271
5218+ - monadlist # #6271
52195219 - non-empty-sequence
52205220 - numeric-limits # #6326/closed
52215221 - oo-prototypes # #6326/closed
@@ -5223,7 +5223,7 @@ packages:
52235223 - opentelemetry-extra # #6326/closed
52245224 - opentelemetry-lightstep # #6326/closed
52255225 - opentelemetry-wai # #6326/closed
5226− - path # 6271
5226+ - path # #6271
52275227 - pattern-arrows
52285228 - planb-token-introspection # #6326/closed
52295229 - pointedlist # #6326/closed
@@ -7439,7 +7439,6 @@ expected-test-failures:
74397439 - cpio-conduit # Test file not in tarball https://github.com/da-x/cpio-conduit/issues/1
74407440 - crypto-pubkey # https://github.com/vincenthz/hs-crypto-pubkey/issues/23
74417441 - doctest
7442− - doctest-discover # 0.1.0.9 https://github.com/karun012/doctest-discover/issues/22
74437442 - ghc-events # https://github.com/haskell/ghc-events/issues/70
74447443 - gitlab-haskell # https://github.com/commercialhaskell/stackage/issues/6088
74457444 - hspec-core # https://github.com/commercialhaskell/stackage/issues/6291
@@ -7694,7 +7693,6 @@ expected-benchmark-failures:
76947693 # Compilation failures
76957694 - OrderedBits # 0.0.2.0
76967695 - aeson-combinators # 0.0.5.0
7697− - cipher-blowfish # 0.0.11 https://github.com/vincenthz/hs-crypto-cipher/issues/46
76987696 - cmark-gfm # https://github.com/kivikakk/cmark-gfm-hs/issues/5
76997697 - cryptohash # 0.11.9 https://github.com/vincenthz/hs-cryptohash/pull/43
77007698 - incremental-parser # 0.5.0.2
etc/commenter/src/lib.rs+17−13
@@ -41,20 +41,20 @@ impl VersionTag {
4141
4242 fn version(&self) -> &str {
4343 match self {
44− VersionTag::Manual(s) => &s,
45− VersionTag::Auto(s) => &s,
44+ VersionTag::Manual(s) => s,
45+ VersionTag::Auto(s) => s,
4646 }
4747 }
4848}
4949
5050pub fn outdated() {
5151 let mut all: Vec<String> = vec![];
52− let disabled = handle(false, |_loc, lines| {
52+ let versioned = handle(false, |_loc, lines| {
5353 all.extend(lines);
5454 vec![]
5555 });
5656 let mut map: BTreeMap<String, VersionTag> = BTreeMap::new();
57− for DisabledPackage { package, version } in disabled {
57+ for VersionedPackage { package, version } in versioned {
5858 map.insert(package, VersionTag::Manual(version));
5959 }
6060 let mut support: BTreeMap<(String, String), BTreeSet<(String, String)>> = BTreeMap::new();
@@ -107,7 +107,7 @@ pub fn outdated() {
107107 let latest = latest_version(&package);
108108 if version != latest {
109109 let max = 3;
110− let dependents_stripped = dependents.len().checked_sub(max).unwrap_or(0);
110+ let dependents_stripped = dependents.len().saturating_sub(max);
111111 let dependents = dependents
112112 .into_iter()
113113 .take(max)
@@ -177,34 +177,38 @@ enum State {
177177 Done,
178178}
179179
180−struct DisabledPackage {
180+struct VersionedPackage {
181181 package: String,
182182 version: String,
183183}
184184
185−fn parse_disabled_package(s: &str) -> Option<DisabledPackage> {
185+fn parse_versioned_package(s: &str) -> Option<VersionedPackage> {
186186 if let Some(caps) = regex!(r#"- *([^ ]+) < *0 *# *([\d.]+)"#).captures(s) {
187187 let package = caps.get(1).unwrap().as_str().to_owned();
188188 let version = caps.get(2).unwrap().as_str().to_owned();
189− Some(DisabledPackage { package, version })
189+ Some(VersionedPackage { package, version })
190+ } else if let Some(caps) = regex!(r#"- *([^ ]+) *# *([\d.]+)"#).captures(s) {
191+ let package = caps.get(1).unwrap().as_str().to_owned();
192+ let version = caps.get(2).unwrap().as_str().to_owned();
193+ Some(VersionedPackage { package, version })
190194 } else {
191195 None
192196 }
193197}
194198
195−fn handle<F>(write: bool, mut f: F) -> Vec<DisabledPackage>
199+fn handle<F>(write: bool, mut f: F) -> Vec<VersionedPackage>
196200where
197201 F: FnMut(Location, Vec<String>) -> Vec<String>,
198202{
199203 let path = "build-constraints.yaml";
200204 let mut new_lines: Vec<String> = vec![];
201− let mut disabled_packages: Vec<DisabledPackage> = vec![];
205+ let mut versioned_packages: Vec<VersionedPackage> = vec![];
202206
203207 let mut state = State::LookingForLibBounds;
204208 let mut buf = vec![];
205209 for line in read_lines(path).map(|s| s.unwrap()) {
206− if let Some(disabled_package) = parse_disabled_package(&line) {
207− disabled_packages.push(disabled_package);
210+ if let Some(versioned_package) = parse_versioned_package(&line) {
211+ versioned_packages.push(versioned_package);
208212 }
209213
210214 match state {
@@ -275,7 +279,7 @@ where
275279 file.flush().unwrap();
276280 }
277281
278− disabled_packages
282+ versioned_packages
279283}
280284
281285enum Location {