Skip to content

docs: explain Go schema and status handling - #43

Closed
imbajin wants to merge 2 commits into
masterfrom
fix/doc-go-schema-status
Closed

imbajin wants to merge 2 commits into
masterfrom
fix/doc-go-schema-status

Conversation

@imbajin

@imbajin imbajin commented Oct 11, 2026 •

Copy link
Copy Markdown

The Go guide lists edge-label creation without explaining that nil or empty properties are valid, and its Gremlin example reads data after checking only err. Explain the schema contract and check both HTTP and Gremlin status before using results in the English and Chinese guides.

Paired code PR: ASF #807. Coordinate the documentation merge with the code PR.

Validation: unchanged link validator, diff checks, bilingual example compilation and a fresh latest strict build passed with cached Hugo 0.165 Extended, Python 3.13 and WebP tools. Generated pages contain the new guidance and status checks. Independent documentation review passed.

Validation baseline: this topic includes the already merged ASF documentation #515, which fixes the pinned-module check with a cold Hugo cache. The org master has not yet picked it up. The upstream PR will contain only these guide changes.

Use native quiet output for the module graph.
Exercise warm and empty Hugo caches in CI.
Add real cold-cache and strict-output regressions.
@github-actions

Copy link
Copy Markdown

@codecov-ai-reviewer review

@coderabbitai

coderabbitai Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 42 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: b2ffc490-cfb7-47a6-8e73-c3c55be8b463

📥 Commits

Reviewing files that changed from the base of the PR and between 6181c87 and b5f27d1.


📒 Files selected for processing (5)
  • .github/workflows/hugo.yml
  • content/cn/docs/quickstart/client/hugegraph-client-go.md
  • content/en/docs/quickstart/client/hugegraph-client-go.md
  • scripts/test_update_oink.py
  • scripts/update_oink.py

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

- document nil and empty edge label properties
- distinguish HTTP status from Go request errors
- check HTTP and Gremlin status in the example
@imbajin
imbajin force-pushed the fix/doc-go-schema-status branch from 99310ec to b5f27d1 Compare October 11, 2026 22:47
@imbajin imbajin closed this Oct 12, 2026
@imbajin
imbajin deleted the fix/doc-go-schema-status branch October 12, 2026 10:24
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