Skip to content

Remove Simple Queries, JS Traversal docs and non-Rest DocuBlocks #14171

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

Merged
merged 9 commits into from
May 26, 2021

Conversation

Simran-B
Copy link
Contributor

@Simran-B Simran-B commented May 11, 2021

Scope & Purpose

Remove old documentation about features that are deprecated since 3.4.0

Also remove all non-Rest DocuBlocks (mostly unused, inlined a few in docs repo).

  • 🔨 simplification
  • 📖 CHANGELOG entry made

Backports:

  • Forwardports required for: 3.9/devel

Related Information

Testing & Verification

  • This change is a trivial rework / code cleanup without any test coverage.
  • The behavior in this PR was manually tested

@Simran-B Simran-B changed the title Remove Simple Queries and JS Traversal docs Remove Simple Queries, JS Traversal docs and non-Rest DocuBlocks May 12, 2021
@Simran-B Simran-B requested a review from joerg84 May 25, 2021 09:17
@Simran-B Simran-B marked this pull request as ready for review May 25, 2021 09:17
Copy link
Contributor

@KVS85 KVS85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comments.

@KVS85
Copy link
Contributor

KVS85 commented May 25, 2021

@KVS85 KVS85 requested a review from gnusi May 26, 2021 16:50
@KVS85
Copy link
Contributor

KVS85 commented May 26, 2021

@KVS85 KVS85 self-requested a review May 26, 2021 17:26
Copy link
Contributor

@KVS85 KVS85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@KVS85 KVS85 merged commit 1cc7352 into 3.8 May 26, 2021
@KVS85 KVS85 deleted the documentation/remove-old-docs branch May 26, 2021 19:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants