-
Notifications
You must be signed in to change notification settings - Fork 1
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
Hiding in-active endpoints: #263 #277
Hiding in-active endpoints: #263 #277
Conversation
Release notes
|
JaCoCo model module code coverage report - scala 2.13.11
|
JaCoCo agent module code coverage report - scala 2.13.11
|
JaCoCo reader module code coverage report - scala 2.13.11
|
JaCoCo server module code coverage report - scala 2.13.11
|
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.
Looks good.
Just to solve the questions.
Mind that in case of conflicts (other endpoints merged) the list can be edited on either side.
createPartitioningEndpointV1, | ||
postPartitioningEndpointV2, | ||
// postPartitioningEndpointV2, | ||
patchPartitioningAdditionalDataEndpointV2, |
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.
Do you guys know, if we use this endpoint in Agent now? @salamonpavel @lsulak
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 don't think so, PATCH wasn't introduced into the agent.
Agent is using this endpoint: POST /api/v2/writeAdditionalData
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.
actually perhaps we should adjust the agent to work with this PATCH because otherwise we got incompatibility in 0.3.0 between agent and server
Release notes
|
c433c8a
to
b27a7fa
Compare
Release notes:
Closes
#263