Skip to content

Conversation

@gusinacio
Copy link
Contributor

Signed-off-by: Gustavo Inacio [email protected]

@gusinacio gusinacio force-pushed the gustavo/impl-tap-rav-traits branch 2 times, most recently from 0360089 to 2618448 Compare February 10, 2025 16:34
@gusinacio gusinacio force-pushed the gustavo/impl-tap-manager-traits branch from 0a49cd2 to f98ceaa Compare February 10, 2025 17:35
@gusinacio gusinacio changed the title refactor: implement receipt traits for horizon refactor: implement rav traits for horizon Feb 10, 2025
Base automatically changed from gustavo/impl-tap-manager-traits to main February 10, 2025 20:01
@gusinacio gusinacio force-pushed the gustavo/impl-tap-rav-traits branch from 2618448 to f90e157 Compare February 10, 2025 21:17
@gusinacio gusinacio force-pushed the gustavo/impl-tap-rav-traits branch from f90e157 to 7dbb32f Compare February 10, 2025 21:19
@gusinacio gusinacio marked this pull request as ready for review February 10, 2025 21:20
@gusinacio gusinacio enabled auto-merge (squash) February 11, 2025 13:06
@gusinacio gusinacio merged commit db93587 into main Feb 11, 2025
10 checks passed
@gusinacio gusinacio deleted the gustavo/impl-tap-rav-traits branch February 11, 2025 13:11
@coveralls
Copy link

coveralls commented Feb 13, 2025

Pull Request Test Coverage Report for Build 13263583625

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 160 of 187 (85.56%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall first build on gustavo/impl-tap-rav-traits at 79.567%

Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/tap-agent/src/tap/context/rav.rs 123 150 82.0%
Totals Coverage Status
Change from base Build 13263546850: 79.6%
Covered Lines: 7858
Relevant Lines: 9876

💛 - Coveralls

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.

5 participants