-
Notifications
You must be signed in to change notification settings - Fork 108
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
Remove GitHub docs in favor of organization level configuration #2210
Conversation
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
Codecov Report
@@ Coverage Diff @@
## main #2210 +/- ##
=======================================
Coverage ? 76.57%
Complexity ? 530
=======================================
Files ? 214
Lines ? 6667
Branches ? 543
=======================================
Hits ? 5105
Misses ? 1438
Partials ? 124 Continue to review full report at Codecov.
|
docs/configuration.md
Outdated
default**. | ||
## Default Values | ||
|
||
The default configuration allows users to get quickly up and running without having to configure anything. This provides |
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.
nit: reads better I think
The default configuration allows users to get quickly up and running without having to configure anything. This provides | |
The default configuration allows users to quickly get up and running without having to configure anything. This provides |
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.
Updated
Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com>
SonarCloud Quality Gate failed. |
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
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
I ended up being able to add the link to HIP in the organization level config. So no changes necessary here for that. |
* Add a issue template config.yml since discussions link is project specific * Add documentation on use of passwords * Fix a line rendering error in deployment diagram * Remove GitHub docs in favor of organization level configuration Signed-off-by: Steven Sheehy <steven.sheehy@hedera.com> Signed-off-by: Ian Jungmann <ian.jungmann@hedera.com>
Detailed description:
Which issue(s) this PR fixes:
Special notes for your reviewer:
CONTRIBUTING.md PR still up for review.
Checklist