Updated outdated README as discussed in a GCI task, including updating features list and adding a link to build instructions.
Details
Diff Detail
- Repository
- R224 KDE Connect
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
This is good. Thank you!
I have added a few comments for making the English sound a little more natural, as well as some little fixes
README.md | ||
---|---|---|
13 | "already existing" is redundant -- I would just change it to "over your WiFi network" or even just "over WiFi" | |
16β17 | I don't know if we need to add a comment about distro support, but I would change the addition to something like "Any disto which supports Qt 5" | |
20 | These typos were already here: But I would actually reword this to: (I prefer "pair both devices" vs. "pair it to your computer" because it avoids mirroring "your computer" in both sentences. It sounds more natural to me) | |
29 | I would leave this sentence as it originally was. "might" means something like "Oh, well, we don't really know" but "should" means "We expect that it will work even though we don't officially support it" (And I think it does work?) | |
59 | Don't you have to specify that we are channel #kdeconnect on freenode? I know almost nothing about IRC π |
I found something else we could change, but all the changes look good to me. Can you update the diff and I will merge it? See inline.
README.md | ||
---|---|---|
23 | I don't think this paragraf is relevant anymore: we already provide an appindicator and Nautilus integration in our repo. Also probably GSConnect is more mature than indicator-kdeconnect nowadays. Can you remove it? |