-
Notifications
You must be signed in to change notification settings - Fork 4
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
Add more context to the README #29
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some additional suggestions:
- The Phylum website already talks about Birdcage
- There is a logo used there that could/should also be used here
- A link to https://www.phylum.io/defend-developers could be included here somewhere
- Complete the last two items on the Community Standards "checklist"
- Contributing guide
- Issue templates
- Some badges might help to make things look "shinier"
- Here is the rendered form:
- And here is a copy/paste form:
[![GitHub](https://img.shields.io/github/license/phylum-dev/birdcage)][license]
[![GitHub issues](https://img.shields.io/github/issues/phylum-dev/birdcage)][issues]
[![Contributor Covenant](https://img.shields.io/badge/Contributor%20Covenant-2.1-4baaaa.svg)][CoC]
[![Discord](https://img.shields.io/discord/1070071012353376387?logo=discord)][discord_invite]
[license]: https://github.com/phylum-dev/birdcage/blob/main/LICENSE
[issues]: https://github.com/phylum-dev/birdcage/issues
[CoC]: https://github.com/phylum-dev/birdcage/blob/main/CODE_OF_CONDUCT.md
[discord_invite]: https://discord.gg/Fe6pr5eW6p
First time I saw this logo now and it's amazing. Should definitely go in the repo too. |
Co-authored-by: Christian Dürr <102963075+cd-work@users.noreply.github.com>
assets/Birdcage.png
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is 50% scale from the original because it looks better in the README
I could have scaled it by using an <img />
tag with height
or width
, but then I would have to use an absolute path to the image. It seemed simpler to scale the image manually so that I can just use the markdown ![]()
syntax with a relative path.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
One typo and a few suggestions.
Co-authored-by: Charles Coggins <maxrake@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Update some language and add details about the history of Birdcage and the technologies that are used.
See the rendered version