-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
[Docs] Split readme md #3405
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
[Docs] Split readme md #3405
Conversation
@dotnet-policy-service agree |
@martincostello Could you please review it? |
I've been travelling for work this week. This is in my TODO pile until I have the time to look at it properly on a computer. |
Could we delete the equivalent content in README in this PR too please? Then we won't have the content in two places. It can be as simple as leaving the headings in place with a simple "See here" link to each section that links to the new page. We can make it look nicer later on. Otherwise LGTM. |
Done |
5412b84
to
44fb768
Compare
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.
Thank you!
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #3405 +/- ##
==========================================
+ Coverage 83.65% 93.55% +9.90%
==========================================
Files 84 110 +26
Lines 3291 3834 +543
Branches 579 719 +140
==========================================
+ Hits 2753 3587 +834
+ Misses 538 247 -291
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Fix link to documentation broken by #3405.
Fix link to documentation broken by #3405.
Pull Request
The issue or feature being addressed
Details on the issue fix or feature implementation
Preview: https://github.com/peter-csala/Swashbuckle.AspNetCore/tree/3365-split-readme-md
Additional stuff:
#
suffixes because they are not needed and they were used inconsistenly<h3>
tags under the Redoc to markdown syntaxOtherwise the content remain intact.
The next PR will improve the formatting of the markdown files.