Skip to content

Conversation

@aritkulova
Copy link
Contributor

@aritkulova aritkulova commented Aug 5, 2025

  • Since this PR suggests a bug fix, the tests have been added and the coverage is 100%.
  • Since this PR introduces a new feature, the update has been discussed in an Issue or with the team.
  • This PR is just a minor change, like a typo fix.

@aritkulova aritkulova self-assigned this Aug 5, 2025
@codecov
Copy link

codecov bot commented Aug 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (dev@5dc1097). Learn more about missing BASE report.
⚠️ Report is 1 commits behind head on dev.

Additional details and impacted files
@@          Coverage Diff           @@
##             dev     #153   +/-   ##
======================================
  Coverage       ?   99.94%           
======================================
  Files          ?       53           
  Lines          ?     3698           
  Branches       ?      371           
======================================
  Hits           ?     3696           
  Misses         ?        2           
  Partials       ?        0           

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@aritkulova aritkulova marked this pull request as ready for review August 8, 2025 14:31
@Arvolear Arvolear changed the base branch from master to dev August 15, 2025 11:19
@Arvolear Arvolear merged commit c78f1e3 into dev Aug 15, 2025
2 checks passed
@Arvolear Arvolear deleted the feat/add-bitcoin-libs branch August 15, 2025 11:19
Arvolear added a commit that referenced this pull request Aug 15, 2025
* Added bitcoin libs (#153)

* added bitcoin libs

* renamed BitcoinHelper to EndianConverter

* refactored natspec

* review fixes
coverage 100%

* BtcTxParser -> TxParser

* updated readme

* refactored tests for txParser

* fixed  parseBlockHeader LE numbers

* refactored endianConverter

* added bytes to bytes conversion in EndianConverter

* cleaned headers data for tests

* review fixes

* package json fix

* refactored parseCuint in TxParser

* refactored naming in EndianConverter

* natspec EndianConverter

* removed convertion function from bytes to uint and vice versa

* covereage for EndianConverter

* refactored _reverseUint128 in EndianConverter

* fixed pragma solidity

* Update ERC7947 interface (#154)

* Update ERC7947 interface

* Fix unit tests

* typo

* added check for insertion attack in TxMerkleProof;
added isBitcoinTransaction and parseCuintMemory to TxParser

* utilized addHexPrefix

* TxMerkleProof now uses txIndex to calculate hashing directions

* fixed typo

* consistency in codestyle

* updated version to 3.2.0

* removed unused test case

* removed excess comment

* fixed natspec

* refactored helpers

* added reference for insertion attack

* update readme

---------

Co-authored-by: Oleh Komendant <44612825+Hrom131@users.noreply.github.com>
Co-authored-by: Artem Chystiakov <artem.ch31@gmail.com>

* remove some contracts

* removed compilation for pragma solidity 0.7.6

* removed bignumber.js from dependencies

* Update AAccountRecovery contract

* update changelog, revert version

---------

Co-authored-by: Yuliia Aritkulova <94910987+aritkulova@users.noreply.github.com>
Co-authored-by: Oleh Komendant <44612825+Hrom131@users.noreply.github.com>
Co-authored-by: Oleh Komendant <oleg02kom@gmail.com>
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.

4 participants