Skip to content
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

tck: HealthResult serialization #10624

Merged
merged 3 commits into from
Mar 20, 2024
Merged

tck: HealthResult serialization #10624

merged 3 commits into from
Mar 20, 2024

Conversation

sdelamo
Copy link
Contributor

@sdelamo sdelamo commented Mar 19, 2024

This pull request adds the HealthResult serialization test introduced in #10526 as a TCK test.

This pull request adds the HealthResult serialization test introduced in #10526 as a TCK test.
@sdelamo sdelamo added the type: improvement A minor improvement to an existing feature label Mar 19, 2024
@sdelamo sdelamo requested a review from timyates March 19, 2024 10:01
@sdelamo sdelamo changed the base branch from 4.4.x to 4.3.x March 20, 2024 10:06
@timyates
Copy link
Contributor

Switching the base to 4.3.x it seems to still see the test-suite-http2-server-tck-netty module which shouldn't be there...

Seeing if an empty commit cleans things up

Copy link

sonarcloud bot commented Mar 20, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@sdelamo sdelamo merged commit 0844f67 into 4.3.x Mar 20, 2024
15 checks passed
@sdelamo sdelamo deleted the healtheresultwithdetails branch March 20, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: improvement A minor improvement to an existing feature
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants