Skip to content

feat: use v2 API if available#59

Merged
robcresswell merged 1 commit into
mainfrom
feat/use-v2-api
Aug 21, 2023
Merged

feat: use v2 API if available#59
robcresswell merged 1 commit into
mainfrom
feat/use-v2-api

Conversation

@robcresswell
Copy link
Copy Markdown
Contributor

This patch adds support for using the new v2 API by default, and falling back to v1. This shouldn't be a breaking change until we remove support for the v1 API

@robcresswell robcresswell self-assigned this Aug 17, 2023
@robcresswell
Copy link
Copy Markdown
Contributor Author

Please review carefully, this was written by an idiot

@robcresswell robcresswell marked this pull request as ready for review August 17, 2023 11:56
@robcresswell robcresswell requested a review from a team as a code owner August 17, 2023 11:56
@robcresswell robcresswell force-pushed the feat/use-v2-api branch 2 times, most recently from 45f4d79 to ed76429 Compare August 18, 2023 10:05
Comment thread pkg/reporter/reporter_test.go
Comment thread pkg/reporter/reporter.go Outdated
Comment thread pkg/reporter/reporter.go Outdated
This patch adds support for using the new v2 API by default, and
falling back to v1. This shouldn't be a breaking change until we remove
support for the v1 API

Signed-off-by: Rob Cresswell <robcresswell@users.noreply.github.com>
Copy link
Copy Markdown
Member

@bradleyjones bradleyjones left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested against Enterprise 4.8 and latest RC and all working good!

@robcresswell robcresswell merged commit a693582 into main Aug 21, 2023
@robcresswell robcresswell deleted the feat/use-v2-api branch August 21, 2023 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants