Skip to content
This repository has been archived by the owner on Aug 21, 2024. It is now read-only.

refactor: make validate_rounding_consts public #1720

Merged

Conversation

OriStarkware
Copy link
Contributor

@OriStarkware OriStarkware commented Mar 26, 2024

This change is Reviewable

@OriStarkware OriStarkware force-pushed the ori/starknet/versioned_constants/os_constants_class/pr10 branch from 3505d4a to f91d8d6 Compare March 26, 2024 12:57
@codecov-commenter
Copy link

codecov-commenter commented Mar 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.47%. Comparing base (8c6475a) to head (f91d8d6).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1720   +/-   ##
=======================================
  Coverage   74.47%   74.47%           
=======================================
  Files          59       59           
  Lines        8536     8536           
  Branches     8536     8536           
=======================================
  Hits         6357     6357           
  Misses       1759     1759           
  Partials      420      420           

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

Copy link
Collaborator

@noaov1 noaov1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:lgtm:

Reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @barak-b-starkware)

@OriStarkware OriStarkware merged commit 59150ce into main Mar 26, 2024
10 checks passed
@OriStarkware OriStarkware deleted the ori/starknet/versioned_constants/os_constants_class/pr10 branch March 26, 2024 13:29
gswirski pushed a commit to reilabs/blockifier that referenced this pull request Jun 26, 2024
* bump rust to `1.76.0`

* wip

* clippy

* update rust ver in ci

* upate dockerfile rust ver

* update ci image
gswirski pushed a commit to reilabs/blockifier that referenced this pull request Jun 26, 2024
…-libs#1719)

ref starkware-libs#1369

- re-export [reth-metrics-derive](https://github.com/paradigmxyz/reth/tree/v0.2.0-beta.4/crates/metrics/metrics-derive) crate from dojo-metrics
- rename metrics -> dojo-metrics crate
- re-export core metrics stuff from dojo-metrics

---

this PR relies on starkware-libs#1720 bcs reth's MSRV is 1.76.0, otherwise can't build reth-metrics-derive.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants