-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
chore: (README.md) Update readme to add bugs, fr #6669
Conversation
Codecov Report
@@ Coverage Diff @@
## main #6669 +/- ##
=======================================
Coverage 73.27% 73.27%
=======================================
Files 213 213
Lines 13195 13195
Branches 2584 2488 -96
=======================================
Hits 9669 9669
- Misses 3332 3362 +30
+ Partials 194 164 -30
Continue to review full report at Codecov.
|
README.md
Outdated
@@ -17,6 +17,11 @@ | |||
</a> | |||
</p> | |||
|
|||
## Reporting Bugs/Feature Requests |
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.
I like these stats! They really help indicate the "health" better than the 658 at the top of the page.
Is this headline needed? It feels a bit out-of-place at the top of the README.
(CC @renebrandel)
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 similar to what iOS is doing so Im curious what @renebrandel thinks as well. May not be needed but I can make the update as needed
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.
I like this 👍 Maybe bring down the header size by 1?
Love it. We should also add this to the CLI repo. |
Sounds good! Ill look to cut a PR to CLI today then for it :) |
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.
Ready to approve, but I added @sammartinez 's change to an H#.
Co-authored-by: Eric Clemmons <eric@smarterspam.com>
This pull request has been automatically locked since there hasn't been any recent activity after it was closed. Please open a new issue for related bugs. Looking for a help forum? We recommend joining the Amplify Community Discord server |
Issue #, if available:
Description of changes:
With the majority of our issues being related to feature requests, I wanted to provide a way to customers to see this on our README. This will show our feature requests currently present along with current bugs and closed issues
Screenshot:

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.