Skip to content

docs: address Oracle post-impl review residuals#25

Merged
yeongseon merged 1 commit into
mainfrom
docs/oracle-followup-links-and-rename
Jul 18, 2026
Merged

docs: address Oracle post-impl review residuals#25
yeongseon merged 1 commit into
mainfrom
docs/oracle-followup-links-and-rename

Conversation

@yeongseon

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #24 addressing defects identified in the Oracle post-implementation review.

Changes

  • CONTRIBUTING.md (lines 1, 159, 160): cubrid-cookbookcubrid-cookbook-python (title + Discussion + Issues links were 404)
  • ROADMAP.md (line 10): Milestones link → cubrid-cookbook-python/milestones
  • SUPPORT_MATRIX.md (line 34): SQLAlchemy 2.0–2.12.0–2.2 (stale; sqlalchemy-cubrid now supports 2.0–2.2)
  • fundamentals/parameterized-queries/: Renamed 04_prepared.py04_parameterized.py (+ .expected) to match the directory name (which was renamed from prepared-statements/ in docs: fix broken links, rename prepared-statements, move PRD, refresh badges #24). Updated README.md link and file docstring.

Out of scope

  • fundamentals/pycubrid/04_prepared.py is a separate file in the numbered 01–14 sequence and was not flagged by Oracle. Left as-is.

Ultraworked with Sisyphus

…name)

- CONTRIBUTING.md: cubrid-cookbook → cubrid-cookbook-python (title + 2 links)
- ROADMAP.md: milestones link → cubrid-cookbook-python
- SUPPORT_MATRIX.md: SQLAlchemy 2.0–2.1 → 2.0–2.2 (stale)
- parameterized-queries/: rename 04_prepared.py → 04_parameterized.py
  (+ .expected) to match directory name; update README link and docstring

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)
Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@yeongseon
yeongseon merged commit 634611b into main Jul 18, 2026
1 check failed
@yeongseon
yeongseon deleted the docs/oracle-followup-links-and-rename branch July 18, 2026 01:57
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.

2 participants