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

refactor: refining codebase; cleanup for initial beta release #127

Merged
merged 10 commits into from
Dec 20, 2024

Conversation

aorumbayev
Copy link
Contributor

@aorumbayev aorumbayev commented Dec 16, 2024

Proposed Changes

  • Refactored project structure and code organization across multiple modules
  • Enhanced type safety with proper annotations and dedicated data classes
  • Initial deprecation documentation at module levels
  • Refining global exports to support legacy vs new project structure

Initial documentation for beta release is to follow in a subsequent PR

Copy link

github-actions bot commented Dec 16, 2024

Coverage

Coverage Report
FileStmtsMissCoverMissing
src/algokit_utils
   _debugging.py1501093%21, 42, 81, 87, 96, 136, 164, 172–174
   application_client.py330%1–11
   config.py891880%26–28, 41, 53, 84–85, 100, 105, 110, 118–123, 156, 158, 160
   deploy.py330%1–10
   logic_error.py330%1–10
src/algokit_utils/_legacy_v2
   _ensure_funded.py71199%100
   _transfer.py67396%13, 76–77
   account.py931386%15–18, 70–74, 109, 126, 156, 159, 208
   application_client.py5327786%60–61, 178, 183, 212, 324, 329–330, 332, 334, 799, 814, 832–835, 925, 965, 977, 990, 1032, 1092–1098, 1102–1107, 1109, 1145, 1152, 1265, 1299, 1313, 1351–1353, 1355, 1365–1422, 1433–1438, 1458–1461
   asset.py82594%10, 28–31
   common.py13192%13
   deploy.py4332295%32–35, 170, 174–175, 193, 249, 339–340, 361, 395, 406–414, 429, 437, 630–631, 655
   network_clients.py73593%77–78, 101–102, 135
src/algokit_utils/accounts
   account_manager.py2063583%106–107, 121–126, 129–134, 147, 176–178, 191–192, 205, 210–214, 217, 240, 346, 437, 502, 507, 525, 528–529, 552
   kmd_account_manager.py71889%57–61, 110, 183, 190
src/algokit_utils/applications
   abi.py1287244%55, 70–71, 77, 83, 85, 87, 91–92, 99–118, 127, 130–136, 144–157, 163–175, 183–194
   app_client.py71721171%56–66, 123, 131, 319–322, 325, 328, 331, 334, 346–349, 352, 355, 358, 361, 373, 382, 391, 396–458, 471–529, 562–564, 573–576, 579–580, 583–586, 589–590, 593–594, 597–598, 611, 673–674, 703, 706, 709, 712, 715, 718, 731, 734, 737, 740, 743, 746, 775–780, 783, 788, 793, 798, 803, 818, 841, 931, 960–961, 980, 982, 988, 1018–1026, 1048–1051, 1054–1057, 1078, 1114, 1127, 1198–1205, 1230, 1242, 1244, 1247, 1251, 1303, 1355, 1373–1374, 1412, 1423–1425, 1429–1453, 1464, 1473–1484, 1492–1497
   app_deployer.py2143982%206, 213, 223–228, 231–235, 273–287, 302–303, 492–505, 515, 520, 526, 532–534, 541, 555, 566–604
   app_factory.py2712790%334, 344, 347, 481, 489, 501, 647, 661–666, 672–673, 692, 717, 728–729, 765, 773–787
   app_manager.py2111493%191, 221–222, 239–244, 265, 301, 314–317, 329, 337
src/algokit_utils/applications/app_spec
   arc32.py95892%195–204
   arc56.py4703692%66–68, 140, 223, 234–235, 283, 296–298, 330, 339, 341, 383, 545, 555, 559, 697, 699, 732–743, 756, 758–759, 775
src/algokit_utils/assets
   asset_manager.py1271291%223–224, 233, 239–254, 264
src/algokit_utils/clients
   algorand_client.py911386%59–60, 69–70, 93–95, 104–105, 199, 215, 228, 238
   client_manager.py1483775%53–60, 66, 90–92, 122, 127–130, 144, 171, 195–198, 220, 244, 256, 270, 286–311, 337, 349, 358
   dispenser_api_client.py831286%126–127, 131–134, 169–171, 190–192
src/algokit_utils/errors
   logic_error.py561279%14, 38, 105–121
src/algokit_utils/models
   account.py63690%81, 86, 107–114
   amount.py1155651%22, 34, 38, 62, 67–70, 74, 77–83, 90, 93–97, 100–104, 107–111, 114–118, 121–125, 128–134, 137–140, 143–149
   application.py42198%8
   state.py36586%51, 55–58
   transaction.py55493%56, 66, 86, 91
src/algokit_utils/protocols
   client.py23674%6–11
src/algokit_utils/transactions
   transaction_composer.py5675291%28–37, 608, 650, 806–807, 833–834, 870, 1020, 1025, 1054–1092, 1212, 1215, 1323, 1325–1326, 1375–1376
   transaction_creator.py72692%119, 124, 129, 134, 139, 144
   transaction_sender.py1451093%82, 204, 245–246, 335–340, 345–346, 370
   utils.py2026866%31, 45, 52, 76, 78, 80, 97, 103, 131–194, 223, 241–243, 249–264, 271–278, 281–288, 299, 367, 370, 375, 379–390
TOTAL602491485% 

Tests Skipped Failures Errors Time
353 0 💤 0 ❌ 0 🔥 3m 13s ⏱️

@aorumbayev aorumbayev force-pushed the prerelease/refactoring branch from 46c2e9e to 3af564c Compare December 18, 2024 14:16
@aorumbayev aorumbayev force-pushed the prerelease/refactoring branch from c9655c0 to eb98689 Compare December 18, 2024 18:00
@aorumbayev aorumbayev force-pushed the prerelease/refactoring branch from eb98689 to 9772e93 Compare December 18, 2024 18:11
@aorumbayev aorumbayev force-pushed the prerelease/refactoring branch from a23c630 to 88306f7 Compare December 18, 2024 21:22
@aorumbayev aorumbayev force-pushed the prerelease/refactoring branch from 88306f7 to 190f2d3 Compare December 18, 2024 21:23
@aorumbayev aorumbayev force-pushed the prerelease/refactoring branch 2 times, most recently from 46edf9f to 72a657d Compare December 20, 2024 18:38
@aorumbayev aorumbayev force-pushed the prerelease/refactoring branch from 72a657d to 53b8c4b Compare December 20, 2024 18:44
@aorumbayev aorumbayev changed the title refactor: refining dataclasses refactor: refining codebase; cleanup for initial beta release Dec 20, 2024
@aorumbayev aorumbayev marked this pull request as ready for review December 20, 2024 18:50
@aorumbayev aorumbayev merged commit 92774f7 into prerelease/ts-feature-parity Dec 20, 2024
4 checks passed
@aorumbayev aorumbayev deleted the prerelease/refactoring branch December 20, 2024 19:42
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.

1 participant