Update README.md #3148
Update README.md #3148
Conversation
Added links to the Discord server and the Matrix room
|
it'd be nicer to make them badges like here |
Added badges [as requested](#3148 (comment)) Added a badge for the npm link as well for consistency with other readme's
|
Not sure how important the zenodo badge is, so I made it go last |
|
keep it first, and if you're gonna add the npm badges add both the lib and cli and both rust badges. |
|
I wasn't sure if I should because there are links for them there already. Should I replace the relevant links with badges? |
README.md
Outdated
| @@ -1,5 +1,8 @@ | |||
| # tree-sitter | |||
|
|
|||
| [![discord][discord]](https://discord.gg/w7nTvsVJhm) | |||
| [![matrix][matrix]](https://matrix.to/#/#tree-sitter-chat:matrix.org) | |||
| [![npm][npm]](https://www.npmjs.com/package/tree-sitter) | |||
There was a problem hiding this comment.
Should link to the tree-sitter-cli package instead.
|
those just link to the readme, not registry page. you can just omit the npm package and limit the scope of this pr to just add chat badges |
keeping the scope of commit only to chat links
The linked readme's have badges to the registry page, wasn't sure if it's better to keep it that way to reduce redundancy, probably doesn't matter either way. |
|
oh right, yeah that's fine as is |
Added links to the Discord server and the Matrix room