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

A lot of small linter fixes #341

Merged
merged 1 commit into from
Jun 3, 2024
Merged

A lot of small linter fixes #341

merged 1 commit into from
Jun 3, 2024

Conversation

slinkydeveloper
Copy link
Contributor

No description provided.

Copy link
Contributor

github-actions bot commented Jun 3, 2024

Test Results

 49 files  ±0   49 suites  ±0   6m 54s ⏱️ -8s
761 tests  - 1  754 ✅  - 1  7 💤 ±0  0 ❌ ±0 
762 runs  ±0  755 ✅ ±0  7 💤 ±0  0 ❌ ±0 

Results for commit 41d57f2. ± Comparison against base commit 1e8d250.

This pull request removes 7 and adds 6 tests. Note that renamed tests count towards both.
dev.restate.sdk.TestSerdesTest ‑ Byte: -12
dev.restate.sdk.TestSerdesTest ‑ Double: 0.21700007454078585
dev.restate.sdk.TestSerdesTest ‑ Float: 0.0073387027
dev.restate.sdk.TestSerdesTest ‑ Integer: 950320261
dev.restate.sdk.TestSerdesTest ‑ Long: 730481082266322252
dev.restate.sdk.TestSerdesTest ‑ Short: 19722
dev.restate.sdk.serde.jackson.JacksonSerdesTest ‑ [Till, Francesco]
dev.restate.sdk.TestSerdesTest ‑ Byte: -69
dev.restate.sdk.TestSerdesTest ‑ Double: 0.9233479486512817
dev.restate.sdk.TestSerdesTest ‑ Float: 0.10561633
dev.restate.sdk.TestSerdesTest ‑ Integer: -483566367
dev.restate.sdk.TestSerdesTest ‑ Long: -5982926446016728924
dev.restate.sdk.TestSerdesTest ‑ Short: -12740

♻️ This comment has been updated with latest results.

@slinkydeveloper slinkydeveloper merged commit 5bba500 into main Jun 3, 2024
4 checks passed
@slinkydeveloper slinkydeveloper deleted the issues/fix-linter branch June 3, 2024 15:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant