propcheck icon indicating copy to clipboard operation
propcheck copied to clipboard

Set proper to the official 1.4 release

Open alfert opened this issue 1 year ago • 11 comments

alfert avatar Mar 16 '24 12:03 alfert

@evnu Any ideas why this fails? It does so also locally on my machine. But I was too long away from Elixir to have a good intuition here.

alfert avatar Mar 16 '24 12:03 alfert

@alfert I think I saw that error with proper some time ago. Currently on mobile, but this might give a hint: https://github.com/proper-testing/proper/blob/a5ae5669f01143b0828fc21667d4f5e344aa760b/src/proper_gen.erl#L361 was there a version bump in proper that we need?

evnu avatar Mar 16 '24 12:03 evnu

@alfert I thought we fixed the eunit error with this: https://github.com/alfert/propcheck/pull/224

evnu avatar Mar 16 '24 12:03 evnu

@alfert ok, so the current proper release on hex.pm is just too old. That's why we used a git revision.

evnu avatar Mar 16 '24 12:03 evnu

This is bad, since we cannot create a proper hex release with dependency on a git hash :-(

alfert avatar Mar 16 '24 19:03 alfert

@alfert so time to ask for a PropEr release then I guess 🙈

evnu avatar Mar 16 '24 20:03 evnu

Seems that there is no new release coming for PropEr any time soon. :-/

evnu avatar Jun 13 '24 14:06 evnu

Yes, feels like Kostis has abandoned PropEr for some time. One way forward could be introducing a fork of PropEr with minor updates (i.e. no new features except for maintenance / upgradability of the Erlang platform). Not sure about it.

alfert avatar Jun 13 '24 15:06 alfert

Forking would mean to release PropEr to hex.pm, I think that's probably more work than we want to have for PropCheck :) Can we vendor-in PropEr and have that as part of this source tree?

evnu avatar Jun 13 '24 15:06 evnu

As a git submodule?

alfert avatar Jun 13 '24 15:06 alfert

If that's possible. This needs to work with mix release as well, I don't know if the Elixir tooling handles vendoring.

evnu avatar Jun 13 '24 16:06 evnu

See https://github.com/alfert/propcheck/pull/230 for a PR to bump the PropEr version.

evnu avatar Apr 03 '25 09:04 evnu