-
Notifications
You must be signed in to change notification settings - Fork 57
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
feat: add increment and add operations to NumberSignal #2694
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2694 +/- ##
==========================================
+ Coverage 92.10% 92.15% +0.04%
==========================================
Files 79 80 +1
Lines 2521 2536 +15
Branches 652 656 +4
==========================================
+ Hits 2322 2337 +15
Misses 152 152
Partials 47 47
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Quality Gate passedIssues Measures |
This ticket/PR has been released with Hilla 24.5.0.alpha13 and is also targeting the upcoming stable 24.5.0 version. |
This will introduce increment event for NumberSignal so that it is processed atomically on server based on the last seen value of NumberSignal without any retries. Also, this refactors the increment method to incrementBy, and also removes the add methods that were introduced in #2694
* fix: adjust increment api and remove add from NumberSignal.ts This will introduce increment event for NumberSignal so that it is processed atomically on server based on the last seen value of NumberSignal without any retries. Also, this refactors the increment method to incrementBy, and also removes the add methods that were introduced in #2694 * remove unnecessary imports and chai-like usage * formatter:format * set the value locally when incrementBy is called * address sonar warnings --------- Co-authored-by: Anton Platonov <[email protected]>
Description
Fixes #2622
Type of change
Checklist
Additional for
Feature
type of change