Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

README: add Platform Support section #981

Merged
merged 4 commits into from
Feb 7, 2025
Merged

Conversation

lance6716
Copy link
Collaborator

@lance6716 lance6716 commented Feb 7, 2025

Copy link
Collaborator

@dveeden dveeden left a comment

Choose a reason for hiding this comment

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

Maybe add a link to https://go.dev/wiki/MinimumRequirements and say that support is limited to OS versions supported by Go. (Only supported Go versions, following the Release Policy on https://go.dev/doc/devel/release ).

Maybe say that this should be able to work on other operating systems, but that this hasn't been tested.

We could do some GOOS=windows go build ... etc in CI to test for incompatibilities. Same for GOARCH=riscv64 etc.

Maybe say that tinygo isn't supported?

README.md Outdated
|------------------|--------------|-----------|----|------------------------------------------------------------------------------------------------------------------------|
| Linux | amd64 | ✅ | ✅ | Check GitHub Actions of this project |
| Linux | s390x | ✅ | ✅ | A daily CI runs on s390x VM, supported by [IBM Z and LinuxOne Community](https://www.ibm.com/community/z/open-source/) |
| Linux | arm64 | ✅ | ❌ | Deployed in a production environment of a user |
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
| Linux | arm64 ||| Deployed in a production environment of a user |
| Linux | arm64 ||| Deployed in a production environment of a user |
| FreeBSD | amd64 ||| Sporadic test by developer |

@lance6716
Copy link
Collaborator Author

@dveeden updated. please check again

Signed-off-by: lance6716 <[email protected]>
@lance6716 lance6716 merged commit 0f8d049 into go-mysql-org:master Feb 7, 2025
15 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.

2 participants