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.

Sign up for GitHub

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

Jump to bottom

Fix lt and lte operators #150

Merged
merged 2 commits into from
Feb 3, 2022
Merged

Fix lt and lte operators #150

merged 2 commits into from
Feb 3, 2022

Conversation

altitude
Copy link
Member

@altitude altitude commented Feb 3, 2022

No description provided.

@altitude altitude requested a review from a team as a code owner February 3, 2022 10:52
@altitude altitude requested a review from gfyrag February 3, 2022 10:52
@altitude
Fix lt and lte operators
4c1a533 
Signed-off-by: Clément Salaün <[email protected]>
@altitude altitude assigned altitude and unassigned altitude Feb 3, 2022
@codecov
Copy link

codecov bot commented Feb 3, 2022 •
edited
Loading

Codecov Report

Merging #150 (5390400) into main (95a9158) will increase coverage by 0.42%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
+ Coverage   73.76%   74.18%   +0.42%     
==========================================
  Files          76       76              
  Lines        2836     2836              
==========================================
+ Hits         2092     2104      +12     
+ Misses        593      581      -12     
  Partials      151      151
Impacted Files Coverage Δ
pkg/ledger/ledger.go 75.23% <ø> (ø)
pkg/core/expr.go 59.47% <100.00%> (+7.84%) ⬆️

Continue to review full report at Codecov.

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

@altitude altitude merged commit a4fe5fa into main Feb 3, 2022
@altitude altitude deleted the fix/expr-lt-lte branch February 3, 2022 14:03
flemzord pushed a commit that referenced this pull request Mar 27, 2023
@paul-nicolas
feat: remove deprecated ledger field for event message (#150)
0e72f6b
flemzord pushed a commit that referenced this pull request Apr 18, 2023
@paul-nicolas
feat: remove deprecated ledger field for event message (#150)
c36376c
flemzord pushed a commit that referenced this pull request Sep 20, 2023
@paul-nicolas
feat: remove deprecated ledger field for event message (#150)
2b31fbb
flemzord pushed a commit that referenced this pull request Sep 25, 2024
@paul-nicolas
feat: remove deprecated ledger field for event message (#150)
9abcb7f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@flemzord flemzord flemzord approved these changes

@gfyrag gfyrag Awaiting requested review from gfyrag gfyrag was automatically assigned from formancehq/core

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@altitude @flemzord