Skip to content

Fix stale path: RANDOM_SCHEMA_TESTING.md moved under ai/test/ - #43

Merged
jnasbyupgrade merged 2 commits into
Postgres-Extensions:masterfrom
jnasbyupgrade:fix-random-schema-doc-path
Aug 10, 2026
Merged

Fix stale path: RANDOM_SCHEMA_TESTING.md moved under ai/test/#43
jnasbyupgrade merged 2 commits into
Postgres-Extensions:masterfrom
jnasbyupgrade:fix-random-schema-doc-path

Conversation

@jnasbyupgrade

Copy link
Copy Markdown
Contributor

Postgres-Extensions/ai#5 relocated it from the repo root to test/ after PR #42 merged.

Postgres-Extensions/ai#5 relocated it from the repo root to test/ after
this PR merged.
@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 3bbe36df-618e-407f-979d-8257c9dd0dfe

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Aug 9, 2026

Copy link
Copy Markdown

Claude finished @jnasbyupgrade's task in 1m 32s —— View job


Review: Fix stale path for RANDOM_SCHEMA_TESTING.md

  • Gather context (diff, changed files)
  • Verify the new path is correct
  • Post review feedback

Verdict: LGTM.

The one-line change in test/CLAUDE.md updates the link target from ../../ai/RANDOM_SCHEMA_TESTING.md to ../../ai/test/RANDOM_SCHEMA_TESTING.md, matching the relocation described in Postgres-Extensions/ai#5.

This is internally consistent with the rest of the file: line 1 already links to ../../ai/test/CLAUDE.md in that same test/ subdirectory, so the corrected RANDOM_SCHEMA_TESTING.md link on line 3 now points to a sibling file in the same directory rather than a stale repo-root path — exactly what you'd expect after that file moved.

No functional code is affected; this is purely a documentation link fix, scoped and minimal. I wasn't able to reach GitHub's API/web from this sandbox to independently confirm the current layout of Postgres-Extensions/ai, but the change is self-consistent with the sibling link already in this file and with the PR description, so I have no concerns.

Nothing else to flag — no tests or CI implications for a doc-link fix like this.

@jnasbyupgrade
jnasbyupgrade merged commit 05beaf1 into Postgres-Extensions:master Aug 10, 2026
4 checks passed
@github-actions

github-actions Bot commented Aug 10, 2026

Copy link
Copy Markdown

Claude finished @jnasbyupgrade's task in 2m 4s —— View job


Code Review: Fix stale path for RANDOM_SCHEMA_TESTING.md

  • Gather context (diff, changed files)
  • Review the change
  • Post final review

Verdict: LGTM.

The change updates test/CLAUDE.md line 3 from ../../ai/RANDOM_SCHEMA_TESTING.md to ../../ai/test/RANDOM_SCHEMA_TESTING.md, per the relocation described in Postgres-Extensions/ai#5.

Observations:

  • Consistent with sibling link: line 1 of the same file already points to ../../ai/test/CLAUDE.md, so the corrected line 3 now resolves to a sibling file in the same ai/test/ directory rather than a stale repo-root path — this is the expected shape after the described move.
  • No other stale references: RANDOM_SCHEMA_TESTING appears exactly once in this repo (test/CLAUDE.md:3), so this one-line fix is complete — nothing else needed updating.
  • Scope: purely a doc-link fix, no functional/SQL code touched, no test or CI impact.
  • Not independently verifiable: I don't have network/API access from this sandbox to confirm the current file layout of Postgres-Extensions/ai (i.e., that test/RANDOM_SCHEMA_TESTING.md actually exists at that path post-move). The change is self-consistent with the file's existing sibling link and the PR description, so I have no concerns, but this fact itself is taken on faith from the PR description rather than confirmed against the ai repo.

Nothing else to flag.

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