-
Notifications
You must be signed in to change notification settings - Fork 10
README updates #389
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
README updates #389
Conversation
updated readme to bring it in line with the latest RC2 updates
Codecov ReportAll modified and coverable lines are covered by tests ✅
❌ Your project status has failed because the head coverage (85.35%) is below the target coverage (90.00%). You can increase the head coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## TDE_REL_17_STABLE #389 +/- ##
==================================================
Coverage 85.35% 85.35%
==================================================
Files 22 22
Lines 2533 2533
Branches 383 383
==================================================
Hits 2162 2162
Misses 297 297
Partials 74 74
🚀 New features to boost your workflow:
|
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.
Does it make sense to have the installation section here at all? Let's maybe have a link to our docs for that? Otherwise, we would have to maintain the same info in two places, which is prone to errors.
removed building for community PG, updated discussions link, updated overview with links to our docs
True, it does not, removed this section as the readme already points to our docs |
left the extension as tde is not a key provider
Updated the pg_tde README based on @Robinyo feedback, removed unnecessary config steps and replaced with links to docs for up to date information, reorganized content and information for clarity and descriptions, improved SEO.