Skip to content

fix(mempool): Log tx hashes as hex instead of bytes#3837

Merged
andynog merged 2 commits intomainfrom
hvanz/mempool-fix-log-tx-hash
Aug 22, 2024
Merged

fix(mempool): Log tx hashes as hex instead of bytes#3837
andynog merged 2 commits intomainfrom
hvanz/mempool-fix-log-tx-hash

Conversation

@hvanz
Copy link
Collaborator

@hvanz hvanz commented Aug 22, 2024

Also, capitalise all first letters of logging messages.


PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

@hvanz hvanz self-assigned this Aug 22, 2024
@hvanz hvanz requested a review from a team as a code owner August 22, 2024 15:18
@hvanz hvanz requested a review from a team August 22, 2024 15:18
Copy link
Collaborator

@melekes melekes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Collaborator

@andynog andynog left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@andynog andynog added this pull request to the merge queue Aug 22, 2024
Merged via the queue into main with commit 2def4c7 Aug 22, 2024
@andynog andynog deleted the hvanz/mempool-fix-log-tx-hash branch August 22, 2024 17:37
mergify bot pushed a commit that referenced this pull request Aug 22, 2024
Also, capitalise all first letters of logging messages.

---

#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments

(cherry picked from commit 2def4c7)

# Conflicts:
#	mempool/clist_mempool.go
#	mempool/reactor.go
hvanz added a commit that referenced this pull request Aug 23, 2024
…3840)

Also, capitalise all first letters of logging messages.

---

#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments
<hr>This is an automatic backport of pull request #3837 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Hernán Vanzetto <15466498+hvanz@users.noreply.github.com>
Co-authored-by: Andy Nogueira <me@andynogueira.dev>
Co-authored-by: hvanz <hernan.vanzetto@gmail.com>
@hvanz hvanz mentioned this pull request Sep 5, 2024
4 tasks
@hvanz hvanz added the logs Anything relating to logging label Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logs Anything relating to logging mempool

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants