Skip to content

Conversation

silverwind
Copy link
Member

@silverwind silverwind commented May 11, 2024

⚠️ BREAKING ⚠️

MSSQL 2017 is the oldest version for which official docker containers exist and we are testing against it. This raises the documented minimum version to that version.

Related: https://endoflife.date/mssqlserver

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 11, 2024
@silverwind silverwind added the pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it! label May 11, 2024
@lunny
Copy link
Member

lunny commented May 12, 2024

Looks like 2016 SP3 has security support until 2025

@silverwind
Copy link
Member Author

silverwind commented May 12, 2024

Looks like 2016 SP3 has security support until 2025

Yes, but only 2017 is available as docker and I think it's better to support only tested versions. It seems 2016 is not available for Linux at all, only 2017+ are.

@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels May 29, 2024
@denyskon
Copy link
Member

denyskon commented Jul 9, 2024

Closing as docs no longer live in this repo. Please open a PR to the docs repo, as we'd need to make this restriction at some point.

@denyskon denyskon closed this Jul 9, 2024
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Oct 7, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged. modifies/docs pr/breaking Merging this PR means builds will break. Needs a description what exactly breaks, and how to fix it!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants