Skip to content
This repository has been archived by the owner on Apr 2, 2024. It is now read-only.

feat(BUX-199): unify logs #499

Merged
merged 15 commits into from
Dec 13, 2023
Merged

feat(BUX-199): unify logs #499

merged 15 commits into from
Dec 13, 2023

Conversation

pawellewandowski98
Copy link
Contributor

#Description

  • Change logger to zerolog

Pull Request Checklist

  • 📖 I created my PR using provided : CODE_STANDARDS
  • 📖 I have read the short Code of Conduct: CODE_OF_CONDUCT
  • 🏠 I tested my changes locally.
  • ✅ I have provided tests for my changes.
  • 📝 I have used conventional commits.
  • 📗 I have updated any related documentation.
  • 💾 PR was issued based on the Github or Jira issue.

@pawellewandowski98 pawellewandowski98 self-assigned this Dec 8, 2023
@pawellewandowski98 pawellewandowski98 requested a review from a team as a code owner December 8, 2023 14:02
@mergify mergify bot added the feature Any new significant addition label Dec 8, 2023
Copy link

codecov bot commented Dec 8, 2023

Codecov Report

Merging #499 (2a42f52) into master (29ad371) will increase coverage by 0.38%.
Report is 3 commits behind head on master.
The diff coverage is 45.37%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #499      +/-   ##
==========================================
+ Coverage   52.99%   53.37%   +0.38%     
==========================================
  Files         110      110              
  Lines       11070    11446     +376     
==========================================
+ Hits         5866     6109     +243     
- Misses       4752     4874     +122     
- Partials      452      463      +11     
Flag Coverage Δ
unittests 53.37% <45.37%> (+0.38%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
beef_tx_bytes.go 94.82% <100.00%> (+4.82%) ⬆️
chainstate/client_options.go 74.66% <100.00%> (ø)
chainstate/transaction.go 94.84% <ø> (ø)
chainstate/transaction_info.go 0.00% <ø> (ø)
client.go 64.26% <100.00%> (+3.17%) ⬆️
client_options.go 69.85% <100.00%> (-2.92%) ⬇️
model_access_keys.go 66.03% <100.00%> (+1.33%) ⬆️
model_bump.go 75.00% <100.00%> (ø)
model_sync_transactions.go 91.35% <100.00%> (+1.65%) ⬆️
model_utxos.go 76.33% <100.00%> (+0.31%) ⬆️
... and 35 more

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 29ad371...2a42f52. Read the comment docs.

action_utxo.go Outdated Show resolved Hide resolved
chainstate/client.go Outdated Show resolved Hide resolved
chainstate/merkle_root.go Outdated Show resolved Hide resolved
client_internal.go Outdated Show resolved Hide resolved
client_internal.go Outdated Show resolved Hide resolved
client_options.go Show resolved Hide resolved
model_incoming_transactions.go Outdated Show resolved Hide resolved
model_incoming_transactions.go Outdated Show resolved Hide resolved
model_incoming_transactions.go Outdated Show resolved Hide resolved
model_paymail_addresses.go Outdated Show resolved Hide resolved
model_sync_transactions.go Outdated Show resolved Hide resolved
bux_suite_test.go Outdated Show resolved Hide resolved
chainstate/client.go Outdated Show resolved Hide resolved
logging/logging.go Show resolved Hide resolved
chainstate/merkle_root.go Outdated Show resolved Hide resolved
chainstate/monitor.go Show resolved Hide resolved
record_tx_strategy_external_incoming_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_internal_incoming_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_outgoing_tx.go Outdated Show resolved Hide resolved
sync_tx_service.go Outdated Show resolved Hide resolved
utils.go Show resolved Hide resolved
@arkadiuszos4chain arkadiuszos4chain self-requested a review December 11, 2023 11:48
model_incoming_transactions.go Outdated Show resolved Hide resolved
model_incoming_transactions.go Outdated Show resolved Hide resolved
model_incoming_transactions.go Outdated Show resolved Hide resolved
model_incoming_transactions.go Outdated Show resolved Hide resolved
model_incoming_transactions.go Outdated Show resolved Hide resolved
paymail.go Outdated Show resolved Hide resolved
paymail.go Outdated Show resolved Hide resolved
record_tx.go Outdated Show resolved Hide resolved
action_utxo.go Outdated Show resolved Hide resolved
@arkadiuszos4chain arkadiuszos4chain self-requested a review December 12, 2023 08:55
model_incoming_transactions.go Outdated Show resolved Hide resolved
record_tx_strategy_internal_incoming_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_internal_incoming_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_internal_incoming_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_internal_incoming_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_internal_incoming_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_outgoing_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_outgoing_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_outgoing_tx.go Outdated Show resolved Hide resolved
record_tx_strategy_outgoing_tx.go Outdated Show resolved Hide resolved
@mergify mergify bot merged commit 6acfb97 into master Dec 13, 2023
10 checks passed
@mergify mergify bot deleted the feat-200-unify-logs branch December 13, 2023 09:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature Any new significant addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants