feat: Travis to GitHub Actions CI#521
Conversation
Test Results (JDK 11)3 001 tests 2 955 ✅ 1m 23s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Test Results (JDK 17)3 001 tests 2 955 ✅ 1m 23s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Test Results (JDK 8)3 001 tests 2 955 ✅ 1m 16s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Code Coverage Report📊 Coverage: % |
msmygit
force-pushed
the
feat/move_to_gha
branch
from
3967618 to
b20d824
Compare
Code Coverage Report📊 Coverage: % |
Integration Test Results (Cassandra 3.11)3 184 tests 3 138 ✅ 1m 29s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Integration Test Results (Cassandra 4.0)3 184 tests 3 138 ✅ 1m 52s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Integration Test Results (DSE 5.1.48)3 182 tests 3 136 ✅ 1m 48s ⏱️ Results for commit 1ef249d. ♻️ This comment has been updated with latest results. |
Integration Test Results (Cassandra 4.1)3 184 tests 3 138 ✅ 2m 24s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Integration Test Results (Cassandra 5.0)3 184 tests 3 138 ✅ 1m 55s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Integration Test Results (DSE 6.9.17)3 182 tests 3 136 ✅ 2m 3s ⏱️ Results for commit 1ef249d. ♻️ This comment has been updated with latest results. |
Code Coverage Report📊 Coverage: % |
Integration Test Results (DSE 6.8.61)3 182 tests 3 136 ✅ 2m 0s ⏱️ Results for commit 1ef249d. ♻️ This comment has been updated with latest results. |
Code Coverage Report📊 Coverage: % |
Code Coverage Report📊 Coverage: % |
msmygit
force-pushed
the
feat/move_to_gha
branch
from
53d0b02 to
3be9b28
Compare
Code Coverage Report📊 Coverage: % |
msmygit
force-pushed
the
feat/move_to_gha
branch
2 times, most recently
from
0d9d941 to
d64b547
Compare
Code Coverage Report📊 Coverage: % |
Code Coverage Report📊 Coverage: % |
Code Coverage Report📊 Coverage: 0% |
| files: ./distribution/target/site/jacoco-aggregate/jacoco.xml | ||
| flags: unittests | ||
| name: codecov-dsbulk | ||
| token: ${{ secrets.CODECOV_TOKEN }} |
There was a problem hiding this comment.
Note
Action Required: Add Codecov Token
- Go to https://codecov.io/ and sign in with your GitHub account
- Add the https://github.com/datastax/dsbulk repository
- Copy the repository token
- Add to DSBulk GitHub repository: Settings → Secrets and variables → Actions → New repository secret
- Name:
CODECOV_TOKEN - Value: [paste token]
Code Coverage Report📊 Coverage: 0% |
Code Coverage Report📊 Coverage: 0% |
Integration Test Results (DSE 6.8.64)3 184 tests 3 138 ✅ 2m 7s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
Integration Test Results (DSE 6.9.23)3 184 tests 3 138 ✅ 2m 12s ⏱️ Results for commit b8710f5. ♻️ This comment has been updated with latest results. |
| assertThat(CodecUtils.parsePoint(null)).isNull(); | ||
| assertThat(CodecUtils.parsePoint("")).isNull(); | ||
| assertThat(CodecUtils.parsePoint("POINT (-1.1 -2.2)")) | ||
| .isEqualTo(new DefaultPoint(-1.1d, -2.2d)); |
There was a problem hiding this comment.
Tip
💡 Code reviewer's assist - This is due to the fact of how the Cassandra Java Driver 4.19.3 changed it, https://github.com/apache/cassandra-java-driver/pull/2052/changes
| } | ||
|
|
||
| public ConvertingCodecAssert<EXTERNAL, INTERNAL> toInternal(INTERNAL internal) { | ||
| assertThat(this.internal) |
There was a problem hiding this comment.
Tip
💡 Code reviewer's assist - This is due to the fact of how the Cassandra Java Driver 4.19.3 changed it, https://github.com/apache/cassandra-java-driver/pull/2052/changes
|
@msmygit Would you mind breaking the Netty and Java driver upgrade (as well as the test fix) out into it's own PR? We'll probably wind up addressing both the GH Actions runner as well as the upgrades for the next release but they are distinct issues and I'd rather have them in distinct PRs so that the historical record is clearer when we look back on this in six months and try to remind ourselves what we were doing and why. :) I can also do all that shuffling around if you'd like... your call. |
Done here, #531 |
… is deprecated warning
Update jacoco to 0.8.8;mockito to 4.11.0 fix with jdk17 runtime
… » IllegalArgument fix
… changed in 4.19.3
msmygit
force-pushed
the
feat/move_to_gha
branch
from
080b2cb to
1984de5
Compare
Code Coverage Report📊 Coverage: 0% |
Code Coverage Report📊 Coverage: 0% |
No description provided.