Skip to content

Add new remove and alter statements#1678

Open
Dhghomon wants to merge 4 commits intomainfrom
more-remove-and-alter
Open

Add new remove and alter statements#1678
Dhghomon wants to merge 4 commits intomainfrom
more-remove-and-alter

Conversation

@Dhghomon
Copy link
Contributor

@vercel
Copy link

vercel bot commented Mar 24, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Ready Ready Preview, Comment Mar 25, 2026 0:13am

Request Review

@netlify
Copy link

netlify bot commented Mar 24, 2026

Deploy Preview for surrealdb-docs failed. Why did it fail? →

Name Link
🔨 Latest commit 3a72401
🔍 Latest deploy log https://app.netlify.com/projects/surrealdb-docs/deploys/69c24aaed000b4000882e5f1

@Dhghomon Dhghomon added the Next release The documentation for this feature is needed for the next release label Mar 24, 2026
Copy link

@claude claude bot left a comment

Choose a reason for hiding this comment

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

LGTM — documentation-only PR adding new ALTER and REMOVE statement pages.

Extended reasoning...

Overview

This PR adds 10 new ALTER statement MDX pages (ACCESS, ANALYZER, API, BUCKET, CONFIG, EVENT, FUNCTION, PARAM, USER, and updates to existing pages), updates sidebar position numbers for existing ALTER pages, adds REMOVE CONFIG to the REMOVE statement, and makes minor syntax corrections in several DEFINE pages ($@name$name, -> @value-> @type).

Security risks

None. This is a documentation-only change with no executable code, auth logic, or data access.

Level of scrutiny

Low. All changes are MDX documentation files following well-established patterns already present in the repository. The railroad diagram JSON structures, syntax blocks, and page layouts are consistent with existing pages. The sidebar position renumbering is mechanical and correct.

Other factors

The single bug flagged by the bug-hunting system (lowercase user on line 17 of alter/user.mdx) is a minor cosmetic nit — ALTER user instead of ALTER USER in prose. This does not affect functionality or correctness of the documented behavior. The overall PR quality is high and the content accurately reflects the linked engine PRs.

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

Labels

Next release The documentation for this feature is needed for the next release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant