Skip to content

Conversation

mpershina
Copy link
Collaborator

@mpershina mpershina commented Aug 1, 2025

anarnold97

This comment was marked as outdated.

Copy link
Collaborator

@anarnold97 anarnold97 left a comment

Choose a reason for hiding this comment

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

Some small nits but otherwise LGTM

@anarnold97 anarnold97 requested a review from ibragins August 13, 2025 13:51
Copy link
Collaborator

@anarnold97 anarnold97 Aug 13, 2025

Choose a reason for hiding this comment

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

this is not correct

[id="installing-downloadable-cli-zip_{context}"]
== Installing the {CLINameTitle} `.zip` file

.Procedure

. Navigate to the link:{DevDownloadPageURL}[{ProductShortName} Download page] and download the OS-specific CLI file or the `src` file:
+
* {ProductShortNameLower}-{ProductVersion}-cli-linux-amd64.zip
* {ProductShortNameLower}-{ProductVersion}-cli-linux-arm64.zip
* {ProductShortNameLower}-{ProductVersion}-cli-darwin-amd64.zip
* {ProductShortNameLower}-{ProductVersion}-cli-darwin-arm64.zip
* {ProductShortNameLower}-{ProductVersion}-cli-windows-amd64.zip
* {ProductShortNameLower}-{ProductVersion}-cli-windows-arm64.zip
* {ProductShortNameLower}-{ProductVersion}-cli-src.zip

. Extract the `.zip` file to the `.kantra` directory inside your `$HOME` directory. The `.zip` file extracts the *mta-cli* binary, along with other required directories and files.

Also, please note Igor's comments on this PR - https://github.com/migtools/mta-documentation/pull/178/files#r2273545164

FYI - @ibragins

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@anarnold97, I think my PTO makes me a bit stupid :D What exatcly is wrong here? The versions or the "extraction" step? Or even both? I checked the versions in the published doc and I don't see the difference . Or have the versions changed in the Download page?

Also, for Igor's comment, yes, we can move the containerless CLI section to the first place. Will think about how to put it the best possible way.

Copy link
Collaborator

Choose a reason for hiding this comment

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

You need to include:

. Extract the `.zip` file to the `.kantra` directory inside your `$HOME` directory. The `.zip` file extracts the `mta-cli` binary, along with other required directories and files.

The .kantra part is essential

Co-authored-by: Andy Arnold <[email protected]>
@migtools migtools deleted a comment from anarnold97 Aug 19, 2025

NOTE: You can also install the CLI for use with Docker on Windows. Note, however, that this is a Developer Preview feature only.
[role="_abstract"]
You can install the {ProductFullName} command-line interface (CLI) on Linux, Windows, or macOS operating systems. You can also install the CLI for use with Docker on Windows. Note, however, that this is a Developer Preview feature only.
Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM

@@ -7,7 +7,10 @@
[appendix,id="contributing-to-mta-development"]
= Contributing to the {ProductShortName} project

To help the {ProductName} cover most application constructs and server configurations, including yours, you can help with any of the following items:
[role="_abstract"]
Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM

@@ -6,7 +6,10 @@
[id="supported-migration-paths_{context}"]
= Supported {ProductName} migration paths

You can run the {ProductFullName} analysis to assess your applications' suitability for migration to multiple target platforms. {ProductShortName} supports the following migration paths:
[role="_abstract"]
You can use the {ProductFullName} to assess your applications' suitability for migration to multiple target platforms.
Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM

@@ -5,7 +5,11 @@
[id="mta-features_{context}"]
= {ProductShortName} features

{ProductFullName} provides the following features to simplify upgrades with more migration paths:
[role="_abstract"]
{ProductFullName} provides the many features that simplify upgrades with multiple migration paths.
Copy link
Collaborator

Choose a reason for hiding this comment

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

LGTM

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