豆豆友情提示:这是一个非官方 GitHub 代理镜像,主要用于网络测试或访问加速。请勿在此进行登录、注册或处理任何敏感信息。进行这些操作请务必访问官方网站 github.com。 Raw 内容也通过此代理提供。
Skip to content

Indicate Go API stability in README#175

Merged
SamMorrowDrums merged 1 commit intomainfrom
williammartin-patch-1
Apr 8, 2025
Merged

Indicate Go API stability in README#175
SamMorrowDrums merged 1 commit intomainfrom
williammartin-patch-1

Conversation

@williammartin
Copy link
Copy Markdown
Collaborator

Description

Some identifiers were recently exported for internal use, but this is not a commitment to a stable API, so let's document that.

Copilot AI review requested due to automatic review settings April 8, 2025 09:04
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR documents that the module's exported Go API should be considered unstable.

  • Added a "Library Usage" section to clarify the potential for breaking changes in the exported API.
  • Updated the README with a disclaimer encouraging users to file an issue if they require a stable API.

Copy link
Copy Markdown
Collaborator

@SamMorrowDrums SamMorrowDrums left a comment

Choose a reason for hiding this comment

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

Perfect

@SamMorrowDrums SamMorrowDrums merged commit 4cf96ab into main Apr 8, 2025
16 checks passed
@SamMorrowDrums SamMorrowDrums deleted the williammartin-patch-1 branch April 8, 2025 09:26
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.

3 participants