Skip to content
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

Documentation typos and updating problem #1677

Closed
raman-m opened this issue Jul 10, 2023 · 0 comments · Fixed by #1678
Closed

Documentation typos and updating problem #1677

raman-m opened this issue Jul 10, 2023 · 0 comments · Fixed by #1678
Assignees
Labels
documentation Needs a documentation update merged Issue has been merged to dev and is waiting for the next release

Comments

@raman-m
Copy link
Member

raman-m commented Jul 10, 2023

Expected Behavior / New Feature

  • One united documentation update of current open PRs
  • Linking PRs and close them as duplicates to decrease current number of open PRs

Actual Behavior / Motivation for New Feature

It is impossible to merge PRs because of lack of permissions, even after approvals by org team members. There is no real management of PRs and issues by owner: PRs stay open, reviewed and approved for a months/years. It is hard to review PRs because of duplication, absence of merging, no updating of feature branches by actual top-commits, etc... These and other problems are below: 👇

  • A significant number of docs update PRs and the number grows constantly
  • It is hard to track and manage docs PRs, duplication of such PRs
  • Tiny/micro PRs take time to approve and merge (current huge repo problem), aka wasting of time
  • Some real PRs contains docs typo fixes and improvements, but it is hard to track these changes of docs, it is hard to compare/update feature branches because of the lack of rights.

Steps to Reproduce the Problem

  1. Go to Pull requests
  2. Filter by label:documentation
  3. Some PRs are related to small changes of text in source code

Specifications

  • Version: latest, 19.0.2
@raman-m raman-m self-assigned this Jul 10, 2023
@raman-m raman-m added accepted Bug or feature would be accepted as a PR or is being worked on documentation Needs a documentation update labels Jul 10, 2023
raman-m added a commit that referenced this issue Sep 25, 2023
* #1676 Update authorization.rst

* #1646 Update bigpicture.rst

* #1628 Update websockets.rst

* #1614 Update README.md

* #1552 Update configuration.rst

* #1547 Update caching.rst

* #1542 Update requestaggregation.rst

* Revert "#1639 Update ClientRateLimitMiddleware.cs"

This reverts commit 996719e.

* #1537 Update routing.rst

* #1520 Update authentication.rst

* #1459 Update building.rst

Fix inline code block

* #1412 Update errorcodes.rst

* #1407 Update headerstransformation.rst

* #1406 Update headerstransformation.rst

Remove extra '!'

* #1320 Update README.md

* Update README.md

Mark classes. Split sentences.

* #1284 Update claimstransformation.rst

* #1232 Update servicediscovery.rst

* #1224 Update loadbalancer.rst

* #1189 Update caching.rst

* Update gettingstarted.rst

Mark up 'net7.0' as code block. Convert script block to PowerShell one. Change the link to NuGet.

* #1386 Update loadbalancer.rst

* Update loadbalancer.rst: Make code snippet shorter

* #1686 Update middlewareinjection.rst

Add section "ASP.NET Core Middlewares and Ocelot Pipeline Builder"

* Update logging.rst

Review RST markup

* Update methodtransformation.rst

Review RST markup

* Update README.md

Sync to "Big Picture" page in Introduction
@raman-m raman-m added merged Issue has been merged to dev and is waiting for the next release and removed accepted Bug or feature would be accepted as a PR or is being worked on labels Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Needs a documentation update merged Issue has been merged to dev and is waiting for the next release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant