docs: update logo and readme #1571
Conversation
There was a problem hiding this comment.
Logo change looks good. @tmihoc there's no change to the readme, did you miss committing a file maybe?
README.md
Outdated
| - Read our [codebase contributor guidelines](https://github.com/canonical/operator/blob/main/HACKING.md) and help improve the codebase | ||
| - Write a charm and publish it on [Charmhub](https://charmhub.io/) | ||
| - Read the [docs](https://ops.readthedocs.io/en/latest/) | ||
| - Read our [Code of conduct](https://ubuntu.com/community/code-of-conduct) join our [chat](https://matrix.to/#/#charmhub-ops:ubuntu.com) and [forum](https://discourse.charmhub.io/) or [open an issue](https://github.com/canonical/operator/issues) |
There was a problem hiding this comment.
| - Read our [Code of conduct](https://ubuntu.com/community/code-of-conduct) join our [chat](https://matrix.to/#/#charmhub-ops:ubuntu.com) and [forum](https://discourse.charmhub.io/) or [open an issue](https://github.com/canonical/operator/issues) | |
| - Read our [Code of conduct](https://ubuntu.com/community/code-of-conduct), join our [chat](https://matrix.to/#/#charmhub-ops:ubuntu.com) and [forum](https://discourse.charmhub.io/), or [open an issue](https://github.com/canonical/operator/issues) |
| - Join our chat: [Mattermost](https://chat.charmhub.io/charmhub/channels/ops) | ||
| - Join our forum: [Discourse](https://discourse.charmhub.io/) | ||
|
|
||
| ### File an issue |
There was a problem hiding this comment.
What's the reason for removing these links? Obviously the doc one needs to be collapsed into the first point rather than pointing to Juju (which probably hasn't been the thing to do for some time...) but I think there's value in having this. If you're reading this on GitHub it's somewhat obvious that there is an issues tab (but it doesn't hurt to be explicit), and on PyPI there is an "issues" link but this spells it out more clearly, and if you only have the dist, then the is the only link in the readme to GitHub I think.
There was a problem hiding this comment.
I think these are moved to the bottom, 3 links on one line; so not totally removed.
There was a problem hiding this comment.
All this info is now given more compatible direct after Next steps. We don't have as many links anymore. (E.g., there's no separate docs vs. API reference.
Done. |
|
/merge |
This PR