Skip to content

Conversation

@jreznot
Copy link
Contributor

@jreznot jreznot commented Aug 9, 2018

I've added the CUBA CLI migrations to make cuba-cli util available on the sdkman. For now, we distribute ZIP archive for Linux / Windows / Mac OS and its is platform specific archive.

version = "1.0.1",
url = "https://bintray.com/cuba-platform/tools/download_file?file_path=cuba-cli%2F1.0.1%2Fcuba-cli-1.0.1-linux.zip",
platform = Linux64
).insert()
Copy link
Member

@marc0der marc0der Aug 9, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you validate all your versions too please? Version and List[Version] both have extension methods for both insert() and validate(), so you could wrap all your versions up in a list and then call insert().validate() on the collection. Here's an example:
https://github.com/sdkman/sdkman-db-migrations/blob/master/src/main/scala/io/sdkman/changelogs/JavaMigrations.scala#L295

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@marc0der marc0der merged commit 6806d88 into sdkman:master Aug 9, 2018
@marc0der
Copy link
Member

marc0der commented Aug 9, 2018

I've made some minor tweaks: I've changed the name to be cuba, not cuba-cli. I've also made 1.0.1 the default version in the candidate.

@jreznot
Copy link
Contributor Author

jreznot commented Aug 9, 2018

Thanks, what is next required for installation through sdk ?

@marc0der
Copy link
Member

marc0der commented Aug 9, 2018

Simply run the following:

$ sdk update

# open a new terminal

$ sdk ls cuba
$ sdk install cuba
$ cuba-cli
...

@marc0der
Copy link
Member

marc0der commented Aug 9, 2018 via email

@jreznot
Copy link
Contributor Author

jreznot commented Aug 9, 2018

You are so fast! Already figured out my problem and removed the message right before your answer.

@marc0der
Copy link
Member

marc0der commented Aug 9, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants