8000 feat: check if normalize markets are enabled in `marketmap` by aljo242 · Pull Request #754 · skip-mev/connect · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Mar 24, 2025. It is now read-only.

feat: check if normalize markets are enabled in marketmap #754

Merged
merged 8 commits into from
Sep 18, 2024
Merged

10000 feat: check if normalize markets are enabled in marketmap #754

merged 8 commits into from
Sep 18, 2024

Conversation

aljo242
Copy link
Contributor
@aljo242 aljo242 commented Sep 17, 2024

Closes CON-1748

@codecov Codecov
Copy link
codecov bot commented Sep 17, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 57.03%. Comparing base (04c8ba5) to head (84ddc0b).
Report is 1 commits behind head on main.

Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #754      +/-   ##
==========================================
+ Coverage   57.00%   57.03%   +0.02%     
==========================================
  Files         208      208              
  Lines       11630    11632       +2     
==========================================
+ Hits         6630     6634       +4     
+ Misses       4396     4395       -1     
+ Partials      604      603       -1

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@aljo242 aljo242 merged commit 66aac54 into main Sep 18, 2024
17 checks passed
@aljo242 aljo242 deleted the feat/enabled-check branch September 18, 2024 17:14
aljo242 pushed a commit that referenced this pull request Dec 10, 2024
feat: check if normalize markets are enabled in marketmap (#754)
aa8a605
aljo242 pushed a commit that referenced this pull request Dec 10, 2024
chore: backport #754 (#853)
23f1def
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@technicallyty technicallyty technicallyty approved these changes

@Eric-Warehime Eric-Warehime Awaiting requested review from Eric-Warehime

@wesl-ee wesl-ee Awaiting requested review from wesl-ee

@zrbecker zrbecker Awaiting requested review from zrbecker

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
@aljo242 @technicallyty
0