Update README.md #683
Update README.md #683
Conversation
Added gettext dependency
ryonakano
left a comment
ryonakano
left a comment
There was a problem hiding this comment.
Thanks for this! Would you sort the list alphabetically? Otherwise LGTM.
Sorted dependencies alphabetically and added some clarity around how to use them + build the application. This was helpful for me when starting out
LuminousHustler
left a comment
LuminousHustler
left a comment
There was a problem hiding this comment.
As a noob myself, I found these small updates/clarifications helpful when starting out
| Run `meson` to configure the build environment and then `ninja` to build | ||
| To build the application locally, clone this repo to your machine. Then, cd to the directory in terminal and run the following commands to configure the build environment and build the application |
There was a problem hiding this comment.
Please keep the original sentence because we use it entirely in our repositories, sorry!
| ## Building and Installation | ||
|
|
||
| You'll need the following dependencies: | ||
| You'll need the following dependencies. Use your package manager (apt) to install them: |
There was a problem hiding this comment.
Please keep the original sentence for distro-agnostic, i.e. not all people who read this README use elementary OS.
There was a problem hiding this comment.
No prob! Makes sense
LuminousHustler
left a comment
LuminousHustler
left a comment
There was a problem hiding this comment.
I think we're good now, I clicked the "+" button beside the original lines which I think reverts my additions but I'm not entirely sure. Here goes nothing!
2 participants
Added gettext dependency