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

Rust 1.74: move lints to Cargo.toml #1162

Merged

Conversation

giladchase
Copy link
Collaborator

@giladchase giladchase commented Nov 22, 2023

commit-id:4f026e58


Stack:

⚠️ Part of a stack created by spr. Do not merge manually using the UI - doing so may have unexpected results.


This change is Reviewable

@giladchase giladchase force-pushed the pr/giladchase/gilad/small-refactor-rust-1.74/4f026e58 branch from 2559ef0 to 2b352bb Compare November 22, 2023 13:22
@codecov-commenter
Copy link

codecov-commenter commented Nov 22, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5037f35) 67.27% compared to head (2b352bb) 67.27%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1162   +/-   ##
=======================================
  Coverage   67.27%   67.27%           
=======================================
  Files          52       52           
  Lines        6742     6742           
  Branches     6742     6742           
=======================================
  Hits         4536     4536           
  Misses       1791     1791           
  Partials      415      415           

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

Copy link
Collaborator

@dorimedini-starkware dorimedini-starkware 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 4 of 4 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @elintul and @noaov1)

@giladchase giladchase merged commit f9bb0c1 into main Nov 22, 2023
15 checks passed
gswirski pushed a commit to reilabs/blockifier that referenced this pull request Jun 26, 2024
* Refactor torii to use proxy server

* Setup dynamic graphql routing
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