Documentation: Fix broken links to Doctrine ORM

[amotl]

Just maintenance.

[coderabbitai]

Walkthrough

Documentation references across multiple files are updated from Doctrine version 3.0 to 3.6, including anchor links and external documentation URLs. A minor PHP syntax correction (missing comma) is applied in connect.rst, and the build version is incremented to 2.2.0.

Changes

Cohort / File(s)	Summary
Documentation anchor and reference updates docs/appendices/compatibility.rst, docs/appendices/data-types.rst, docs/index.rst	Updated Doctrine DBAL and ORM documentation anchor references from version 3.0 to 3.6 across multiple files
External reference updates and syntax fix docs/connect.rst	Updated external Doctrine DBAL and ORM documentation links from 3.0 to 3.6; added missing comma in PHP example array definition
Version bump docs/build.json	Incremented build version from 2.1.2 to 2.2.0

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 A version leap, from three-oh to three-six we go,
Links updated, syntax gently mended,
Build incremented, docs now rendered bright,
Small changes ripple, making all things right! 📚✨

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	❓ Inconclusive	The description 'Just maintenance' is vague and generic, using a non-descriptive term that doesn't convey meaningful information about the specific changes made (link updates and version bump).	Provide a more descriptive summary of the maintenance work, such as 'Update Doctrine documentation links from 3.0 to 3.6 and bump docs build version to 2.2.0'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly describes the main change: fixing broken links to Doctrine ORM documentation. This directly reflects the primary content of the changeset, which updates documentation references from version 3.0 to 3.6.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-broken-links

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5a88e12 and 3363b66.

📒 Files selected for processing (5)

• docs/appendices/compatibility.rst
• docs/appendices/data-types.rst
• docs/build.json
• docs/connect.rst
• docs/index.rst

🧰 Additional context used

🧠 Learnings (2)

📚 Learning: 2025-11-21T18:32:35.719Z

Learnt from: amotl
Repo: crate/crate-dbal PR: 122
File: src/Crate/DBAL/Driver/PDOCrate/PDOConnection.php:0-0
Timestamp: 2025-11-21T18:32:35.719Z
Learning: Documentation update: docs/index.rst now explicitly states CrateDB implements eventual consistency and does not support traditional ACID transactions (commit 171116851). Preference in this repo is to document this limitation rather than raise runtime exceptions in transaction methods.

Applied to files:

• docs/index.rst
• docs/connect.rst

📚 Learning: 2025-11-19T23:30:55.289Z

Learnt from: amotl
Repo: crate/crate-dbal PR: 122
File: src/Crate/DBAL/Driver/PDOCrate/PDOConnection.php:48-48
Timestamp: 2025-11-19T23:30:55.289Z
Learning: In crate-dbal's PDOConnection.php, PDO::ATTR_STATEMENT_CLASS is set using a bare class string (PDOStatement::class) rather than the standard PDO array format ([ClassName::class, [args]]) because PDOCrateDB is a custom PDO implementation that accepts a class name string directly for this attribute.

Applied to files:

• docs/index.rst

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build docs on ubuntu-latest

🔇 Additional comments (5)

docs/build.json (1)

4-4: LGTM!

The version bump from 2.1.2 to 2.2.0 is appropriate for documentation updates.

docs/appendices/compatibility.rst (1)

76-76: Documentation link is accessible and correctly updated.

The URL for the Doctrine DBAL 3.6 query builder reference returns HTTP 200, confirming the link update from version 3.0 to 3.6 is valid and aligns with the PR's objective to fix broken documentation references.

docs/appendices/data-types.rst (1)

114-114: The updated Doctrine ORM documentation link is accessible and correctly configured.

The link at line 114 (https://www.doctrine-project.org/projects/doctrine-orm/en/3.6/reference/basic-mapping.html) returns HTTP 200, confirming it is reachable. The version update from 3.0 to 3.6 is properly applied and aligns with the PR's objective to fix broken documentation references.

docs/connect.rst (1)

107-107: Documentation links are all accessible.

All four updated documentation links (DBAL 3.6 and ORM 3.6) return HTTP 200 and are working correctly. The version updates from 3.0 to 3.6 align with the PR's objective to fix broken links.

docs/index.rst (1)

37-37: Documentation links are properly updated and accessible.

Both the DBAL and Doctrine ORM documentation links have been successfully updated to version 3.6 and are accessible (HTTP 200). The changes align with the PR's objective to fix broken links.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

docs/connect.rst (1)

67-72: Fix the PHP syntax error in the array definition.

Line 69 is missing a comma after 'user' => 'crate', which will cause a parse error.

🔎 Proposed fix

    $params = array(
        'driverClass' => 'Crate\DBAL\Driver\PDOCrate\Driver',
-       'user' => 'crate'
+       'user' => 'crate',
        'host' => 'localhost',
        'port' => 4200
    );

📜 Review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5a88e12 and 3363b66.

📒 Files selected for processing (5)

• docs/appendices/compatibility.rst
• docs/appendices/data-types.rst
• docs/build.json
• docs/connect.rst
• docs/index.rst

🧰 Additional context used

🧠 Learnings (2)

📚 Learning: 2025-11-21T18:32:35.719Z

Learnt from: amotl
Repo: crate/crate-dbal PR: 122
File: src/Crate/DBAL/Driver/PDOCrate/PDOConnection.php:0-0
Timestamp: 2025-11-21T18:32:35.719Z
Learning: Documentation update: docs/index.rst now explicitly states CrateDB implements eventual consistency and does not support traditional ACID transactions (commit 171116851). Preference in this repo is to document this limitation rather than raise runtime exceptions in transaction methods.

Applied to files:

• docs/index.rst
• docs/connect.rst

📚 Learning: 2025-11-19T23:30:55.289Z

Learnt from: amotl
Repo: crate/crate-dbal PR: 122
File: src/Crate/DBAL/Driver/PDOCrate/PDOConnection.php:48-48
Timestamp: 2025-11-19T23:30:55.289Z
Learning: In crate-dbal's PDOConnection.php, PDO::ATTR_STATEMENT_CLASS is set using a bare class string (PDOStatement::class) rather than the standard PDO array format ([ClassName::class, [args]]) because PDOCrateDB is a custom PDO implementation that accepts a class name string directly for this attribute.

Applied to files:

• docs/index.rst

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build docs on ubuntu-latest

🔇 Additional comments (5)

docs/build.json (1)

4-4: LGTM!

The version bump from 2.1.2 to 2.2.0 is appropriate for documentation updates.

docs/appendices/compatibility.rst (1)

76-76: Documentation link is accessible and correctly updated.

The URL for the Doctrine DBAL 3.6 query builder reference returns HTTP 200, confirming the link update from version 3.0 to 3.6 is valid and aligns with the PR's objective to fix broken documentation references.

docs/appendices/data-types.rst (1)

114-114: The updated Doctrine ORM documentation link is accessible and correctly configured.

The link at line 114 (https://www.doctrine-project.org/projects/doctrine-orm/en/3.6/reference/basic-mapping.html) returns HTTP 200, confirming it is reachable. The version update from 3.0 to 3.6 is properly applied and aligns with the PR's objective to fix broken documentation references.

docs/connect.rst (1)

107-107: Documentation links are all accessible.

All four updated documentation links (DBAL 3.6 and ORM 3.6) return HTTP 200 and are working correctly. The version updates from 3.0 to 3.6 align with the PR's objective to fix broken links.

docs/index.rst (1)

37-37: Documentation links are properly updated and accessible.

Both the DBAL and Doctrine ORM documentation links have been successfully updated to version 3.6 and are accessible (HTTP 200). The changes align with the PR's objective to fix broken links.