Skip to content

Conversation

@CraigMacomber
Copy link
Contributor

Description

I don't find the documentation for lowestMinVersionForCollab particularly informative, nor do I think the approach it describes is the best one.

This attempts to establish a practical process for how it should be set which minimizes the support obligation each major will incur while keeping the required ability to collab with all previously supported major versions.

Reviewer Guidance

The review process is outlined on this wiki page.

Copilot AI review requested due to automatic review settings November 6, 2025 03:20
@github-actions github-actions bot added area: runtime Runtime related issues base: main PRs targeted against main branch labels Nov 6, 2025
Copy link
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 updates the minimum supported version for collaboration from 1.0.0 to 1.4.0 to align with the actual supported version policy. The change reflects that versions below 1.4.0 were no longer supported at the time of the 2.0 release.

  • Bumped lowestMinVersionForCollab constant from "1.0.0" to "1.4.0"
  • Enhanced documentation explaining the rationale for supporting 1.4.0 as the minimum version
  • Updated comments to provide guidance for future major version releases

@agarwal-navin
Copy link
Contributor

@scottn12 FYI. I think it makes to set this to 1.4. I am not aware that we had promised any more support than this with 2.0.

@scottn12
Copy link
Contributor

scottn12 commented Nov 7, 2025

@scottn12 FYI. I think it makes to set this to 1.4. I am not aware that we had promised any more support than this with 2.0.

Yeah I think we had considered 1.4 previously as well. It should be fine and there really shouldn't be any clients that exist today older than 1.4

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: runtime Runtime related issues base: main PRs targeted against main branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants