Skip to content
This repository has been archived by the owner on Feb 5, 2025. It is now read-only.

Fixer mixer #748

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fixer mixer #748

wants to merge 2 commits into from

Conversation

crStiv
Copy link

@crStiv crStiv commented Dec 4, 2024

Closes: No specific issue number.


This PR:

  • Fixes multiple typos and minor wording inconsistencies in the documentation for asynchronous data retrieval.
  • Updates the SQL migration file to correct a typographical error in comments.

This PR does not:

  • Introduce new functionality or features.
  • Modify any core logic or add significant changes to the repository.

Key places to review:

  • doc/async-retrieval.md:
    • Corrected redundant word usage, e.g., "best effort" to "best-effort."
    • Adjusted grammar for clarity in multiple sections, e.g., “effect” changed to “affect” and “need” corrected to “needed.”
    • Updated phrasing for better readability, e.g., "increment sequentially" instead of "an increment sequentially."
  • migrations/postgres/V100__drop_leaf_payload.sql:
    • Typo fix in comment: “unnecesssary” corrected to “unnecessary.”

How to test this PR:

No specific tests are required as changes are limited to documentation and comments. Review for accuracy and clarity.


Complete the following items before merging:

  • Verify that the documentation edits align with intended meaning.
  • Confirm that the SQL migration file comment changes are accurate and maintain readability.

Allow edits by maintainers.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant