Aaron S. Hawley

Results 82 comments of Aaron S. Hawley

Looks like [1.15.7](https://github.com/clutchski/coffeelint/compare/v1.15.1...v1.15.7) was administrivia and one bug fix: - Fix reporters printing warnings when quiet flag set #563 And then it looks like [1.16.0](https://github.com/clutchski/coffeelint/compare/v1.15.7...v1.16.0) was more administravia after updating...

The tags could be these branches - 0.8.3 8ec6b89 - 0.8.4 11d38b2 And this commit on 0.9.0 branch - 0.9.0-beta1 3a3a176 But I'm just guessing

Agreed. Lines 58 and 59 reference `.bashrc` and `bash` itself. What shell do you use?

I haven't looked at your code, but are you using ScalaCheck 1.14.3? There was a defect with command shrinking with Scala 2.12, see #447.

I ran your code example, but it didn't compile. Is this something that depends on #632? ```scala override def shrinkCommand: Shrink[Command] = Shrink { ... ```

Yes, `retryUntil` is meant to specify a condition on generated values like `suchThat`, but with less restriction. It's risky and should be used with caution, as Rickard pointed out in...

Yes, there is room for improvement with shrinking. It's simplistic, as is. You're suggesting 1) the shrink should be able to run in the other direction, not just to zero,...

> we know that the test succeeds on `n / 2` and fails on `n`, thus there exists some smallest value `k` between those two extremes for which the test...

I was moving on to imaginging what the implementation would need to be, but I'm probably incorrect. Is there any documentation for how this is done in Rust's QuickCheck?

There's a [serializability test](https://github.com/typelevel/scalacheck/blob/9aa8d60/jvm/src/test/scala/org/scalacheck/SerializabilitySpecification.scala) in ScalaCheck that works. Ironically, the import you needed was removed from that test in [d8f1322](https://github.com/typelevel/scalacheck/commit/d8f13227bf80b6644a240ba8b845afaba57b24c5#diff-d4a5d71f728a8709230a83a7fc095322). I know forking the JVM was needed for the test...