Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Making README shorter #124

Merged
merged 7 commits into from
Mar 31, 2022
Merged

Making README shorter #124

merged 7 commits into from
Mar 31, 2022

Conversation

jloleysens
Copy link
Contributor

@jloleysens jloleysens commented Mar 30, 2022

  • Move Roadmap to just above contributing so that Getting started is on top
  • Added link to releases page
  • Tightened up language a lot

Preview: https://github.com/animo/aries-cli/tree/Making-README-shorter

README.md Show resolved Hide resolved
passing the `--verbose` (or `-v`) flag to get more information about what the CLI is
doing. We also support stacking verbosity up to three levels for debug logs: `-vv` and `-vvv`.
If you are getting started with the tool we recommend enabling informational logs by
passing the `--verbose` (or `-v`).

## Common examples
Copy link
Member

Choose a reason for hiding this comment

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

Not too sure if the common example should be included in the README.md

Copy link
Contributor Author

@jloleysens jloleysens Mar 30, 2022

Choose a reason for hiding this comment

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

i think it is a natural flow for new users after getting set up which is nice. Ideally we'd send them to the docs that'll be more comprehensive after that. In short, I think it'd be nice to keep for now, then possibly link out to docs and shorten or remove in future. WDYT?


The `--toolbox` flag makes sure the invite has an alias as Toolbox, sets auto accept to true and sets the toolbox as admin for the invite.

### Create an invitation for the toolbox
Copy link
Member

Choose a reason for hiding this comment

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

Same for this

@jloleysens
Copy link
Contributor Author

I'm going to go ahead and merge this for now @blu3beri @anagoessens happy to take follow up requests for changes if you feel strongly about it!

@jloleysens jloleysens merged commit 84dd48d into main Mar 31, 2022
@jloleysens jloleysens deleted the Making-README-shorter branch March 31, 2022 10:34
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.

3 participants