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

use starknet_api crate #710

Merged
merged 3 commits into from
Jun 30, 2023
Merged

use starknet_api crate #710

merged 3 commits into from
Jun 30, 2023

Conversation

juanbono
Copy link
Collaborator

@juanbono juanbono commented Jun 29, 2023

Use starknet_api crate

Description

Closes #708
This PR replaces lambda_starknet_api by starknet_api because now it is available in crates.io.

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.

@juanbono juanbono self-assigned this Jun 29, 2023
@juanbono juanbono linked an issue Jun 29, 2023 that may be closed by this pull request
@codecov-commenter
Copy link

Codecov Report

Merging #710 (4b8ddab) into main (3af21ba) will increase coverage by 0.25%.
The diff coverage is 92.30%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #710      +/-   ##
==========================================
+ Coverage   92.11%   92.36%   +0.25%     
==========================================
  Files          52       52              
  Lines       11360    11792     +432     
==========================================
+ Hits        10464    10892     +428     
- Misses        896      900       +4     
Impacted Files Coverage Δ
src/definitions/block_context.rs 100.00% <ø> (ø)
crates/starknet-contract-class/src/lib.rs 79.56% <85.71%> (-0.64%) ⬇️
.../api/contract_classes/deprecated_contract_class.rs 96.85% <100.00%> (-0.08%) ⬇️
src/storage/errors/storage_errors.rs 100.00% <100.00%> (ø)

... and 8 files with indirect coverage changes

@Oppen Oppen added this pull request to the merge queue Jun 30, 2023
Merged via the queue into main with commit 5361f71 Jun 30, 2023
@juanbono juanbono deleted the use_starknet_api branch July 1, 2023 16:20
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.

Use starknet_api dependency
5 participants