Skip to content

doc - #96

Merged
visualfc merged 1 commit into
goplus:mainfrom
visualfc:doc2
Aug 4, 2026
Merged

doc#96
visualfc merged 1 commit into
goplus:mainfrom
visualfc:doc2

Conversation

@visualfc

@visualfc visualfc commented Aug 4, 2026

Copy link
Copy Markdown
Member

No description provided.

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Review Summary

Documentation-only change to README.md. Verified against the repo:

  • Version range Go 1.21 ~ 1.26 matches go.mod (go 1.21) and the CI matrix in .github/workflows/go.yml (1.21.x1.26.x).
  • The Go ABI section's Go1.21+ / Go1.23+ references stay within the documented range — no contradiction.
  • Splitting the old flat list into Go Version / Compilers / Platforms preserves all prior content; nothing dropped.
  • The added LLGo link (https://github.com/goplus/llgo) is a legitimate sibling goplus project — no security concern.

No blocking issues. One optional cosmetic nit noted inline.

Comment thread README.md Outdated

[![Build Status](https://github.com/goplus/reflectx/workflows/Go/badge.svg)](https://github.com/goplus/reflectx/workflows/Go/badge.svg)


Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Optional nit: this introduces a second consecutive blank line before ### Go Version. It renders identically to a single blank, but is inconsistent with the single-blank spacing used elsewhere — consider removing it.

@visualfc
visualfc merged commit 8d5331d into goplus:main Aug 4, 2026
18 checks passed
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