-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Bump release version to 1.13 #4666
Conversation
Signed-off-by: p-wysocki <przemyslaw.wysocki@intel.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The PR itself looks good to me. Will approve it after code freeze. Thank you @p-wysocki for driving ONNX 1.13 release!
@@ -7,6 +7,6 @@ | |||
namespace ONNX_NAMESPACE { | |||
|
|||
// Represents the most recent release version. Updated with every release. | |||
constexpr const char* LAST_RELEASE_VERSION = "1.12.0"; | |||
constexpr const char* LAST_RELEASE_VERSION = "1.13.0"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.13.0rc1
we shall make it 1.13.0 when the release is validated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR is bumping version number in the main branch first and so I think 1.13.0 is correct here. See old releases: #3973, #4209.
By contrast, the release manager will further cut the release branch and then manually change the version number in https://github.com/onnx/onnx/blob/main/VERSION_NUMBER with XXXrc1.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also need to make change to https://github.com/onnx/onnx/blob/main/VERSION_NUMBER
@liqunfu Do you mean https://github.com/onnx/onnx/pull/4666/files#diff-22aa3cdc2dc9bec750e99a529ea7ac3d2642c71212cb3d8843265916ea7ffc18L1? That file has been changed. |
Signed-off-by: p-wysocki <przemyslaw.wysocki@intel.com> Signed-off-by: p-wysocki <przemyslaw.wysocki@intel.com> Co-authored-by: G. Ramalingam <grama@microsoft.com>
Signed-off-by: p-wysocki <przemyslaw.wysocki@intel.com> Signed-off-by: p-wysocki <przemyslaw.wysocki@intel.com> Co-authored-by: G. Ramalingam <grama@microsoft.com>
Signed-off-by: p-wysocki przemyslaw.wysocki@intel.com
Description
PR is a part of an ongoing 1.13.0 release. It's based on #4209.