Skip to content

Small English grammar/pluralization fixes #2651

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.

Sign up for GitHub

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

Open
dflock wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Small English grammar/pluralization fixes #2651

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions packages/docs/data-loaders/index.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ app.mount('#app')

## Quick start

There are different data loaders implementation, the simplest one is the [Basic Loader](./basic/) which always reruns data fetching. A more efficient one, is the [Colada Loader](./colada/) which uses [@pinia/colada](https://github.com/posva/pinia-colada) under the hood. In the following examples, we will be using the _basic loader_.
There are different data loader implementations, the simplest one is the [Basic Loader](./basic/) which always reruns data fetching. A more efficient one, is the [Colada Loader](./colada/) which uses [@pinia/colada](https://github.com/posva/pinia-colada) under the hood. In the following examples, we will be using the _basic loader_.
Copy link
Contributor

@coderabbitai coderabbitai bot Mar 6, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

This sentence still has two grammar issues.

Line 43 still reads awkwardly because of the comma splice after “implementations” and the extra comma after “one”. Consider splitting it into two sentences, e.g. “There are different data loader implementations. The simplest one is …”

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@packages/docs/data-loaders/index.md` at line 43, The sentence beginning
"There are different data loader implementations, the simplest one is the [Basic
Loader]..." contains a comma splice and an extra comma; split it into two
sentences and remove the extra comma so it reads like: "There are different data
loader implementations. The simplest one is the [Basic Loader]..." and keep the
rest of the clause about Colada Loader and the example choice unchanged.


Loaders are [composables](https://vuejs.org/guide/reusability/composables.html) defined through a `defineLoader` function like `defineBasicLoader` or `defineColadaLoader`. They are _used_ in the component `setup` to extract the needed information.

Expand Down Expand Up @@ -92,4 +92,4 @@ On top of that, you are free to _reuse_ the returned composable `useUserData` in

## Why Data Loaders?

Data fetching is the most common need for a web application. There are many ways of handling data fetching, and they all have their pros and cons. Data loaders are a way to streamline data fetching in your application. Instead of forcing you to choose between different libraries, data loaders provide a consistent way to manage data fetching in your application no matter the underlying library or strategy you use.
Data fetching is the most common needs for a web application. There are many ways of handling data fetching, and they all have their pros and cons. Data loaders are a way to streamline data fetching in your application. Instead of forcing you to choose between different libraries, data loaders provide a consistent way to manage data fetching in your application no matter the underlying library or strategy you use.
Copy link
Contributor

@coderabbitai coderabbitai bot Mar 6, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

This pluralization is still incorrect.

Line 95 should be either “Data fetching is the most common need for a web application” or “Data fetching is one of the most common needs for a web application.”

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@packages/docs/data-loaders/index.md` at line 95, Fix the pluralization in the
sentence currently reading "Data fetching is the most common needs for a web
application" by changing it to either "Data fetching is the most common need for
a web application" or "Data fetching is one of the most common needs for a web
application" (choose one phrasing to keep tone consistent with the rest of the
document); update the sentence in the paragraph that begins with "Data fetching
is the most common needs for a web application."