search

reitzig / sdkman-for-fish

Adds support for SDKMAN! to fish
MIT License
280 stars 13 forks source link

Test documented installation path #36

Closed reitzig closed 3 years ago

reitzig commented 4 years ago

Since the README tells people to use fisher to install, we should test that.

While I can not reproduce locally, #32 and #33 suggest that there is some issue. As per https://github.com/jorgebucaran/fisher/issues/557, fisher does not seem to have a general problem, either.

Add a (nightly?) build for different platforms that uses fisher to install sdkman-for-fish (and then SDKMAN? through that our routine?).

reitzig commented 3 years ago

Fixed on dev: CI now tests that the current fisher can install the current sdkman-for-fish. Future versions will have been run through that test.

In fact, even the Cucumber tests run after installing the code under test with fisher, so we also confirm the installation is functional.