-
Notifications
You must be signed in to change notification settings - Fork 91
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
Cleanup of the docs folder and Contributing #5619
Conversation
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
docs/contributing/contributing-code/contributing-code-control-plane/configSettings.md
Show resolved
Hide resolved
...ntributing/contributing-code/contributing-code-first-commit/first-commit-00-prerequisites.md
Show resolved
Hide resolved
Test Results2 748 tests ±0 2 741 ✔️ ±0 1m 56s ⏱️ -1s Results for commit 4db4a9b. ± Comparison against base commit 2228304. This pull request removes 2 and adds 2 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
c827e68
to
8c0c98d
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
docs/contributing/contributing-code/contributing-code-control-plane/logging.md
Outdated
Show resolved
Hide resolved
docs/contributing/contributing-code/contributing-code-control-plane/logging.md
Outdated
Show resolved
Hide resolved
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 grammarlied the PR :) other than the comments I left, LGTM...
docs/contributing/contributing-code/contributing-code-building/README.md
Outdated
Show resolved
Hide resolved
docs/contributing/contributing-code/contributing-code-cli/README.md
Outdated
Show resolved
Hide resolved
docs/contributing/contributing-code/contributing-code-cli/README.md
Outdated
Show resolved
Hide resolved
docs/contributing/contributing-code/contributing-code-control-plane/README.md
Outdated
Show resolved
Hide resolved
docs/contributing/contributing-code/contributing-code-control-plane/configSettings.md
Outdated
Show resolved
Hide resolved
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
This PR is a major overhaul of the content in our docs folder. I found some loose threads as part of this work so I pulled on them, which is why there are so many changes. Major changes: - Reorganize of docs/contributing - Rewrite of contributing.md - Lots of NEW content (how to contribute, how to file issues, how to file pull requests, how to work on tests) - Lots of old/unused cruft removed
b5572c1
to
4db4a9b
Compare
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Getting a linter failure I haven't seen before, so retrying tests. I'm trying to repro locally, I don't believe my PR caused this. |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
One more time! Previous failures caused by: golangci/golangci-lint#3877 |
Radius functional test overview
Click here to see the list of tools in the current test run
Test Status⌛ Building Radius and pushing container images for functional tests... |
Description
This PR is a major overhaul of the content in our docs folder. I found some loose threads as part of this work so I pulled on them, which is why there are so many changes.
Major changes:
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:
Auto-generated summary
🤖 Generated by Copilot at c827e68
Summary
🔧📚🔒
This pull request removes unused or outdated files, enables authentication for the RP service, improves the contributor documentation, and fixes some build and debug issues. It also adds a new file with instructions on how to build the code and containers.
Walkthrough