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

Improve dependency requirements at README.md #43

Merged
merged 1 commit into from
May 11, 2022

Conversation

fadelkon
Copy link
Contributor

When installing, I prefer to have the steps for my platform together, instead of having the steps grouped by dependencies and in each one, "on mac, do this; on debian, do that". Also allows for copy-pasting.

I see no advantage of how it was until now, as there's no explanation of why each dependency is needed. This can be added at the "generic" list I added. It could read "summary", for example.

I wish I'm also correcting some confunsion:

  • were you recommending to install adb via brew/apt, or via android "platform tools"?
  • was brew bundle an alternative to install everything or just an optional step of many more?

When installing, I prefer to have the steps for my platform together, instead of having the steps grouped by dependencies and in each one, "on mac, do this, on debian, do that"
@rchatterjee rchatterjee merged commit b5d00e8 into stopipv:main May 11, 2022
tomrist pushed a commit that referenced this pull request Mar 10, 2023
Improve dependency requirements at README.md
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