-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Added 7.1.0 changelog #3262
Added 7.1.0 changelog #3262
Conversation
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.
LGTM
@laurazard PTAL
----- | ||
### Upgrade Notes | ||
- Bumped minimum engine API version to 1.24 | ||
- Bumped default engine API version to 1.44 (Moby 25.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.
If #3261 is planned to get merged for 7.1.0, this needs to be updated.
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.
We were discussing that one, and maybe (given that there may still be some questions remaining there), we should skip that one for now, and leave it for the next release.
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.
Left a couple of nits (just removing periods as I don't think we tend to use them, I might've left them there myself though 😅), but otherwise LGTM.
b489cf0
to
b08d27f
Compare
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.
LGTM!
Signed-off-by: Christopher Petito <[email protected]>
b08d27f
to
4f2a26d
Compare
No description provided.