Skip to content

docs: improve wording and fix minor text issues in installation documentation#2294

Open
DoraC7 wants to merge 1 commit intoqodo-ai:mainfrom
DoraC7:DoraC7-patch-2
Open

docs: improve wording and fix minor text issues in installation documentation#2294
DoraC7 wants to merge 1 commit intoqodo-ai:mainfrom
DoraC7:DoraC7-patch-2

Conversation

@DoraC7
Copy link
Copy Markdown

@DoraC7 DoraC7 commented Mar 28, 2026

Description

Fix minor wording, punctuation, and capitalization issues in the installation documentation.

Changes

  • Corrected Bitbucket capitalization
  • Improved wording for the OpenAI API key description
  • Refined sentence structure for better clarity and readability

Notes

These changes are documentation-only and do not affect runtime behavior.

…stallation docs

Description

Fix minor wording, punctuation, and capitalization issues in the installation documentation.

Changes
Corrected Bitbucket capitalization
Improved wording for the OpenAI API key description
Refined sentence structure for better clarity and readability
Notes

These changes are documentation-only and do not affect runtime behavior.
@qodo-free-for-open-source-projects
Copy link
Copy Markdown
Contributor

Review Summary by Qodo

Fix wording and capitalization in installation documentation

📝 Documentation

Grey Divider

Walkthroughs

Description
• Corrected "BitBucket" capitalization to "Bitbucket" throughout
• Improved OpenAI key description wording for clarity
• Refined sentence structure in environment variables section
• Enhanced readability of configuration documentation
Diagram
flowchart LR
  A["Installation Docs"] -->|"Fix capitalization"| B["BitBucket → Bitbucket"]
  A -->|"Improve wording"| C["OpenAI key description"]
  A -->|"Refine sentences"| D["Environment variables section"]
  B --> E["Updated docs"]
  C --> E
  D --> E
Loading

Grey Divider

File Changes

1. docs/docs/installation/locally.md 📝 Documentation +7/-7

Improve wording and fix capitalization issues

• Changed "BitBucket" to "Bitbucket" in two locations for correct capitalization
• Updated "An OpenAI key" to "An OpenAI API key" for clarity
• Improved wording from "a key for other" to "another" for better grammar
• Restructured GitHub token instruction from "GitHub token, for example" to "GitHub token. For
 example, you can create"
• Refined environment variables documentation with clearer phrasing and better sentence structure
• Changed "expected names and values of environment variables" to "the expected names and values of
 environment variables" for improved flow

docs/docs/installation/locally.md


Grey Divider

Qodo Logo

@qodo-free-for-open-source-projects
Copy link
Copy Markdown
Contributor

qodo-free-for-open-source-projects bot commented Mar 28, 2026

Code Review by Qodo

🐞 Bugs (4) 📘 Rule violations (0) 📎 Requirement gaps (0) 📐 Spec deviations (0)

Grey Divider


Action required

1. Grammar: another models 🐞 Bug ⚙ Maintainability
Description
The OpenAI key bullet says "or another language models", which is grammatically incorrect (singular
"another" with plural "models") and makes the alternative-model option unclear.
Code

docs/docs/installation/locally.md[3]

+1. An OpenAI API key from [here](https://platform.openai.com/api-keys){:target="_blank"}, with access to GPT-4 and o4-mini (or another [language models](../usage-guide/changing_a_model.md), if you prefer).
Evidence
The edited line contains the incorrect phrase "another [language models]".

docs/docs/installation/locally.md[3-3]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
The text uses an ungrammatical phrase: `another [language models]`.

### Issue Context
This is in the prerequisite keys list at the top of the local installation guide.

### Fix Focus Areas
- docs/docs/installation/locally.md[3-3]

### Suggested change
Update to either:
- `... (or other [language models](../usage-guide/changing_a_model.md), if you prefer).`

or
- `... (or another [language model](../usage-guide/changing_a_model.md), if you prefer).`

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


2. Sentence fragment: GitHub token 🐞 Bug ⚙ Maintainability
Description
The personal access token bullet now contains a standalone "GitHub token." sentence fragment, which
breaks the sentence and reduces clarity.
Code

docs/docs/installation/locally.md[4]

+2. A personal access token from your Git platform (GitHub, GitLab, Bitbucket, Gitea) with repo scope. GitHub token. For example, you can create a GitHub token [here](https://github.com/settings/tokens){:target="_blank"}
Evidence
The updated bullet splits the thought into "...with repo scope. GitHub token. For example...",
leaving "GitHub token." as an incomplete sentence.

docs/docs/installation/locally.md[4-4]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`GitHub token.` is a sentence fragment inserted between two complete sentences.

### Issue Context
This is the prerequisites list for running PR-Agent locally.

### Fix Focus Areas
- docs/docs/installation/locally.md[4-4]

### Suggested change
Rewrite as a single coherent sentence, e.g.:
- `2. A personal access token from your Git platform (GitHub, GitLab, Bitbucket, Gitea) with repo scope. For example, you can create a GitHub token [here](https://github.com/settings/tokens){:target="_blank"}.`

(Or similar wording, but remove the standalone `GitHub token.` fragment.)

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Remediation recommended

3. Bitbucket naming inconsistent 🐞 Bug ⚙ Maintainability
Description
This page now uses "Bitbucket" while other installation pages still use "BitBucket", creating
inconsistent terminology across the docs.
Code

docs/docs/installation/locally.md[37]

+- For Bitbucket:
Evidence
The locally installation page uses "Bitbucket" in the edited section, but other installation pages
in the same docs set use "BitBucket", making capitalization inconsistent across the documentation
set.

docs/docs/installation/locally.md[37-37]
docs/docs/installation/index.md[5-10]
docs/docs/installation/pr_agent.md[33-40]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
Docs use both `Bitbucket` and `BitBucket` across installation pages, which is inconsistent.

### Issue Context
`locally.md` was changed to `Bitbucket`, but `installation/index.md` and `installation/pr_agent.md` still use `BitBucket`.

### Fix Focus Areas
- docs/docs/installation/locally.md[37-37]
- docs/docs/installation/index.md[5-10]
- docs/docs/installation/pr_agent.md[33-40]

### Suggested change
Pick one capitalization for the platform name across the docs and apply it consistently (either update the remaining pages to `Bitbucket` or revert this change to match the existing docs style).

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools



Advisory comments

4. Env placeholders unclear 🐞 Bug ⚙ Maintainability
Description
The environment-variable explanation treats "<KEY>=<VALUE>" as a single placeholder, which is less
clear than describing <KEY> and <VALUE> separately as substitution parts of an assignment.
Code

docs/docs/installation/locally.md[62]

+`<TABLE>` refers to a section in a configuration file and `<KEY>=<VALUE>` refers to the key/value pair of a setting in the configuration file.
Evidence
The text states "<KEY>=<VALUE> refers to the key/value pair", while the preceding convention already
shows an assignment form; clarifying <KEY> and <VALUE> separately would better match the
convention and example syntax.

docs/docs/installation/locally.md[60-62]
docs/docs/installation/locally.md[67-72]

Agent prompt
The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
The wording implies `<KEY>=<VALUE>` is one placeholder rather than an assignment with two placeholders.

### Issue Context
This section explains the environment variable naming/value convention.

### Fix Focus Areas
- docs/docs/installation/locally.md[60-62]

### Suggested change
Reword line 62 to clarify substitution, e.g.:
- `` `<TABLE>` refers to a section in a configuration file; `<KEY>` is the setting name and `<VALUE>` is the setting value. ``

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools


Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo


1. An OpenAI key from [here](https://platform.openai.com/api-keys){:target="_blank"}, with access to GPT-4 and o4-mini (or a key for other [language models](../usage-guide/changing_a_model.md), if you prefer).
2. A personal access token from your Git platform (GitHub, GitLab, BitBucket, Gitea) with repo scope. GitHub token, for example, can be issued from [here](https://github.com/settings/tokens){:target="_blank"}
1. An OpenAI API key from [here](https://platform.openai.com/api-keys){:target="_blank"}, with access to GPT-4 and o4-mini (or another [language models](../usage-guide/changing_a_model.md), if you prefer).
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Action required

1. Grammar: another models 🐞 Bug ⚙ Maintainability

The OpenAI key bullet says "or another language models", which is grammatically incorrect (singular
"another" with plural "models") and makes the alternative-model option unclear.
Agent Prompt
### Issue description
The text uses an ungrammatical phrase: `another [language models]`.

### Issue Context
This is in the prerequisite keys list at the top of the local installation guide.

### Fix Focus Areas
- docs/docs/installation/locally.md[3-3]

### Suggested change
Update to either:
- `... (or other [language models](../usage-guide/changing_a_model.md), if you prefer).`

or
- `... (or another [language model](../usage-guide/changing_a_model.md), if you prefer).`

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

1. An OpenAI key from [here](https://platform.openai.com/api-keys){:target="_blank"}, with access to GPT-4 and o4-mini (or a key for other [language models](../usage-guide/changing_a_model.md), if you prefer).
2. A personal access token from your Git platform (GitHub, GitLab, BitBucket, Gitea) with repo scope. GitHub token, for example, can be issued from [here](https://github.com/settings/tokens){:target="_blank"}
1. An OpenAI API key from [here](https://platform.openai.com/api-keys){:target="_blank"}, with access to GPT-4 and o4-mini (or another [language models](../usage-guide/changing_a_model.md), if you prefer).
2. A personal access token from your Git platform (GitHub, GitLab, Bitbucket, Gitea) with repo scope. GitHub token. For example, you can create a GitHub token [here](https://github.com/settings/tokens){:target="_blank"}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Action required

2. Sentence fragment: github token 🐞 Bug ⚙ Maintainability

The personal access token bullet now contains a standalone "GitHub token." sentence fragment, which
breaks the sentence and reduces clarity.
Agent Prompt
### Issue description
`GitHub token.` is a sentence fragment inserted between two complete sentences.

### Issue Context
This is the prerequisites list for running PR-Agent locally.

### Fix Focus Areas
- docs/docs/installation/locally.md[4-4]

### Suggested change
Rewrite as a single coherent sentence, e.g.:
- `2. A personal access token from your Git platform (GitHub, GitLab, Bitbucket, Gitea) with repo scope. For example, you can create a GitHub token [here](https://github.com/settings/tokens){:target="_blank"}.`

(Or similar wording, but remove the standalone `GitHub token.` fragment.)

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant