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

Use Eclipse Temurin + 8/11/17 matrix #695

Merged
merged 4 commits into from Sep 23, 2021
Merged

Use Eclipse Temurin + 8/11/17 matrix #695

merged 4 commits into from Sep 23, 2021

Conversation

jponge
Copy link
Member

@jponge jponge commented Sep 23, 2021

No description provided.

@cescoffier
Copy link
Contributor

The last time I checked, Kotlin was failing. Did we update it recently?

@jponge
Copy link
Member Author

jponge commented Sep 23, 2021

We've had a bump PR merged recently

@jponge jponge added this to the 1.1.0 milestone Sep 23, 2021
@jponge jponge added the ci/cd/build CI/CD pipeline and build label Sep 23, 2021
@jponge
Copy link
Member Author

jponge commented Sep 23, 2021

So Dokka is broken: Kotlin/dokka#2163

@jponge
Copy link
Member Author

jponge commented Sep 23, 2021

@cescoffier I propose to limit the build matrix do 8/11 until the Dokka situation improves /cc @heubeck

@codecov
Copy link

codecov bot commented Sep 23, 2021

Codecov Report

Merging #695 (a73fc77) into main (639ef12) will increase coverage by 0.24%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #695      +/-   ##
============================================
+ Coverage     89.91%   90.16%   +0.24%     
- Complexity     2977     2984       +7     
============================================
  Files           374      374              
  Lines         11770    11770              
  Branches       1476     1478       +2     
============================================
+ Hits          10583    10612      +29     
+ Misses          611      589      -22     
+ Partials        576      569       -7     
Impacted Files Coverage Δ
...erators/multi/builders/BufferItemMultiEmitter.java 90.78% <0.00%> (-1.32%) ⬇️
...tiny/operators/multi/MultiBufferWithTimeoutOp.java 73.94% <0.00%> (-0.85%) ⬇️
...mallrye/mutiny/operators/multi/MultiFlatMapOp.java 85.04% <0.00%> (-0.32%) ⬇️
...mallrye/mutiny/operators/multi/MultiGroupByOp.java 83.25% <0.00%> (+0.44%) ⬆️
...perators/multi/processors/SerializedProcessor.java 84.15% <0.00%> (+0.99%) ⬆️
...smallrye/mutiny/helpers/test/AssertSubscriber.java 91.72% <0.00%> (+1.50%) ⬆️
...subscription/SwitchableSubscriptionSubscriber.java 99.18% <0.00%> (+1.63%) ⬆️
...java/io/smallrye/mutiny/helpers/Subscriptions.java 80.66% <0.00%> (+1.65%) ⬆️
...y/operators/multi/builders/IterableBasedMulti.java 84.04% <0.00%> (+2.12%) ⬆️
...utiny/operators/multi/MultiWindowOnDurationOp.java 70.71% <0.00%> (+5.00%) ⬆️
... and 2 more

@jponge
Copy link
Member Author

jponge commented Sep 23, 2021

...or we can have a Maven profile for Java 17 that disables Dokka

@jponge
Copy link
Member Author

jponge commented Sep 23, 2021

I just disabled Dokka on 17 and enabled plain 17 in Ci/CD

@jponge jponge merged commit cbfbfcf into main Sep 23, 2021
@jponge jponge deleted the ci-temurin branch September 23, 2021 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/cd/build CI/CD pipeline and build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants