Skip to content

Conversation

xerial
Copy link
Member

@xerial xerial commented Aug 6, 2023

No description provided.

@github-actions github-actions bot added bug internal Internal changes (usually non-user facing) labels Aug 6, 2023
@xerial xerial enabled auto-merge (squash) August 6, 2023 05:03
@codecov
Copy link

codecov bot commented Aug 6, 2023

Codecov Report

Merging #3108 (6a1ecdd) into master (1d21a04) will decrease coverage by 0.07%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3108      +/-   ##
==========================================
- Coverage   82.90%   82.84%   -0.07%     
==========================================
  Files         349      349              
  Lines       14700    14700              
  Branches     2443     2412      -31     
==========================================
- Hits        12187    12178       -9     
- Misses       2513     2522       +9     
Files Changed Coverage Δ
...in/scala/wvlet/airframe/http/openapi/OpenAPI.scala 87.50% <ø> (-0.74%) ⬇️
...cala/wvlet/airframe/codec/MessageCodecFinder.scala 91.11% <100.00%> (ø)
...c/main/scala/wvlet/airframe/codec/UnionCodec.scala 75.00% <100.00%> (+2.27%) ⬆️

... and 4 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d21a04...6a1ecdd. Read the comment docs.

@xerial xerial changed the title openapi (internal): Fix flaky tests openapi (internal): Do not use runtime-reflection in UnionCodec to fix flaky tests Aug 6, 2023
@xerial xerial merged commit 9fef8e1 into master Aug 6, 2023
@xerial xerial deleted the openapi-test-fix branch August 6, 2023 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug internal Internal changes (usually non-user facing)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant