Skip to content

snap: update description - #165

Merged
niemeyer merged 2 commits into
canonical:mainfrom
rebornplusplus:snap/metadata
Nov 22, 2024
Merged

niemeyer merged 2 commits into
canonical:mainfrom
rebornplusplus:snap/metadata

Conversation

@rebornplusplus

@rebornplusplus rebornplusplus commented Oct 16, 2024 •

Copy link
Copy Markdown

Resolves #164.

Updates the snap description. Uses the output from chisel help --all instead of chisel --help for details. Removes the usage from snap description.

Comment thread snap/snapcraft.yaml Resolved
@cjdcordeiro cjdcordeiro added the Simple Nice for a quick look on a minute or two label Oct 17, 2024

@letFunny letFunny left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

There is the question of whether we want to change the description of the commands or add more information. In my opinion, that can wait until next cycle where we will have a focus on documentation. For now it seems like an easy win to update the snap to contain the same information that the CLI has.

Thanks Rafid for this UX improvements!

@linostar
linostar self-requested a review October 23, 2024 12:57
@letFunny letFunny added the Polish Refactorings, etc label Nov 8, 2024
Comment thread snap/snapcraft.yaml Resolved

@niemeyer niemeyer left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the changes.

@niemeyer
niemeyer merged commit a09dd15 into canonical:main Nov 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Polish Refactorings, etc Simple Nice for a quick look on a minute or two

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore: update Chisel snap description

5 participants