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

feat: add architecture test which ensure that others modules do not use contract internals #60

Merged
merged 12 commits into from
Dec 28, 2023

Conversation

kamilbaczek
Copy link
Collaborator

No description provided.

@kamilbaczek kamilbaczek marked this pull request as draft December 18, 2023 20:38
@kamilbaczek kamilbaczek changed the title feat: add contracts architecture test feat: add architecture test which ensure that others modules do not use contract internals Dec 20, 2023
@kamilbaczek kamilbaczek marked this pull request as ready for review December 20, 2023 15:58
@kamilbaczek kamilbaczek force-pushed the feature/architecture_tests_chapter_1 branch from 34237a0 to b04fffc Compare December 21, 2023 19:49
@codecov-commenter
Copy link

codecov-commenter commented Dec 22, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2ebcf18) 96.80% compared to head (49f661c) 96.81%.
Report is 3 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #60   +/-   ##
=======================================
  Coverage   96.80%   96.81%           
=======================================
  Files          61       61           
  Lines         532      533    +1     
  Branches        9        9           
=======================================
+ Hits          515      516    +1     
  Misses         14       14           
  Partials        3        3           

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

@kamilbaczek kamilbaczek force-pushed the feature/architecture_tests_chapter_1 branch from 87a223e to 49f661c Compare December 26, 2023 15:07
@kamilbaczek kamilbaczek force-pushed the feature/architecture_tests_chapter_1 branch from 9ce808d to 6df83e4 Compare December 28, 2023 18:01
@kamilbaczek kamilbaczek merged commit c05422d into main Dec 28, 2023
2 checks passed
@kamilbaczek kamilbaczek deleted the feature/architecture_tests_chapter_1 branch December 28, 2023 18:02
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.

3 participants