-
Notifications
You must be signed in to change notification settings - Fork 493
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
Adding changelog for v1.1.0-rc1 #2967
Adding changelog for v1.1.0-rc1 #2967
Conversation
Skipping CI for Draft Pull Request. |
cf123f6
to
cbcbb50
Compare
Gateways now feature a new field that allows references to | ||
implementation-specific parameters, similar to GatewayClass. | ||
|
||
Leading Contributors: @howardjohn |
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.
@howardjohn can you help fill this out?
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.
/approve
CHANGELOG.md
Outdated
The Conformance Reports API and the corresponding test suite have been graduated to GA. The Conformance report API has been expanded with the `mode` field (intended to specify the working mode of the implementation), and the `gatewayAPIChannel` (standard or experimental). The `gatewayAPIVersion` and `gatewayAPIChannel` are now filled in automatically by the suite machinery, along with a brief description of the testing outcome. | ||
The Reports have been reorganized in a more structured way, and the implementations can now add information on how the tests have been run and provide reproduction steps. |
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.
I just noticed you seem to be 80-lining for some lines above, but not these? NBD but if you feel like updating it 🤷
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.
These are just the result of me accepting GitHub suggestions. Once they're all in, I'll update the line length for everything.
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.
@robscott would you want to add anything about the work that @xtineskim completed? It's user facing and it holds alot of meaning for some folks.
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.
Good call, it would be great to mention that if it merges before we release. #2874 is still in progress, but agree that it will be a huge and meaningful update when it merges.
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.
/approve
This release is huge! Excited to get this out and thanks to everyone who contributed!
781fe27
to
195bc9f
Compare
Co-authored-by: Mattia Lavacca <[email protected]>
Co-authored-by: Grant Spence <[email protected]>
Co-authored-by: Arko Dasgupta <[email protected]>
Co-authored-by: Candace Holman <[email protected]>
Co-authored-by: John Howard <[email protected]>
Co-authored-by: Candace Holman <[email protected]>
Co-authored-by: Candace Holman <[email protected]>
Co-authored-by: Gaurav Kumar Ghildiyal <[email protected]>
Co-authored-by: Candace Holman <[email protected]>
Co-authored-by: Frank Budinsky <[email protected]>
Co-authored-by: Frank Budinsky <[email protected]>
Co-authored-by: Flynn <[email protected]>
Co-authored-by: Candace Holman <[email protected]>
8299337
to
2692c74
Compare
2692c74
to
f7f4682
Compare
f7f4682
to
f298424
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.
/approve
/lgtm
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mikemorris, robscott, shaneutt The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/hold |
Waiting for #3019 |
What type of PR is this?
/kind documentation
What this PR does / why we need it:
This adds the changelog for v1.1.0-rc1. By approving this PR, you're approving the release of RC1 based on these changes.
Does this PR introduce a user-facing change?: