Skip to content

Add storetype option to keytool command example for linux#13213

Open
solsticedhiver wants to merge 1 commit intoflutter:mainfrom
solsticedhiver:patch-2
Open

Add storetype option to keytool command example for linux#13213
solsticedhiver wants to merge 1 commit intoflutter:mainfrom
solsticedhiver:patch-2

Conversation

@solsticedhiver
Copy link
Copy Markdown
Contributor

@solsticedhiver solsticedhiver commented Mar 23, 2026

Also specify storetype of JKS for linux example

Description of what this PR is changing or adding, and why:
Correct the linux keytool example and add -storetype JKS to match the windows example

Issues fixed by this PR (if any):

PRs or commits this PR depends on (if any):

Presubmit checklist

  • If you are unwilling, or unable, to sign the CLA, even for a tiny, one-word PR, please file an issue instead of a PR.
  • If this PR is not meant to land until a future stable release, mark it as draft with an explanation.
  • This PR follows the Google Developer Documentation Style Guidelines—for example, it doesn't use i.e. or e.g., and it avoids I and we (first-person pronouns).
  • This PR uses semantic line breaks
    of 80 characters or fewer.

Also specify storetype of JKS for linux example
@solsticedhiver solsticedhiver requested review from a team and sfshaza2 as code owners March 23, 2026 11:55
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request correctly adds the -storetype JKS option to the keytool command example for Linux/macOS, aligning it with the Windows example and ensuring it works correctly with modern Java versions. I have one suggestion to improve the formatting of the command for better readability and consistency.

@sfshaza2
Copy link
Copy Markdown
Contributor

@solsticedhiver, can you address the bot's feedback?

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