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.

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

Contract api #248

Merged
merged 1 commit into from
Nov 13, 2023
Merged

Contract api #248

merged 1 commit into from
Nov 13, 2023

Conversation

jawoznia
Copy link
Collaborator

Impl Extension trait on Contract. This change is required to generate entry_points without requiring user to specify which generics are for which messages.

Unfortunately rust compiler requires fully-qualified path and in current state it's not the most user friendly Api so the primary use case is code generation.

#86

@jawoznia jawoznia requested a review from hashedone October 26, 2023 09:49
@jawoznia jawoznia changed the base branch from main to generate_contract_types_in_sv_module October 26, 2023 09:50
@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Merging #248 (c653e88) into feat/generics_support (9d44792) will increase coverage by 1.14%.
The diff coverage is 87.34%.

@@                    Coverage Diff                    @@
##           feat/generics_support     #248      +/-   ##
=========================================================
+ Coverage                  85.87%   87.02%   +1.14%     
=========================================================
  Files                         24       25       +1     
  Lines                       1657     1726      +69     
=========================================================
+ Hits                        1423     1502      +79     
+ Misses                       234      224      -10     
Files Coverage Δ
sylvia-derive/src/input.rs 93.44% <100.00%> (+0.33%) ⬆️
sylvia-derive/src/interfaces.rs 86.86% <100.00%> (ø)
sylvia-derive/src/message.rs 85.07% <100.00%> (+2.34%) ⬆️
sylvia-derive/src/multitest.rs 97.23% <100.00%> (+0.93%) ⬆️
sylvia-derive/src/remote.rs 100.00% <100.00%> (ø)
sylvia/src/types.rs 80.00% <ø> (ø)
sylvia/tests/api.rs 0.00% <0.00%> (ø)

... and 2 files with indirect coverage changes

📣 Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!

@jawoznia jawoznia force-pushed the contract_api branch 2 times, most recently from 09c3158 to 9bab234 Compare October 26, 2023 10:12
sylvia/tests/api.rs Outdated Show resolved Hide resolved
@jawoznia jawoznia force-pushed the generate_contract_types_in_sv_module branch from 8f840c3 to 25500b7 Compare October 30, 2023 10:39
Base automatically changed from generate_contract_types_in_sv_module to feat/generics_support October 30, 2023 10:58
Copy link
Collaborator

@hashedone hashedone left a comment

Choose a reason for hiding this comment

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

LGTM

@jawoznia jawoznia merged commit ea75996 into feat/generics_support Nov 13, 2023
7 checks passed
@jawoznia jawoznia deleted the contract_api branch November 13, 2023 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants