Skip to content

chore: remove assethub from bouncer#6435

Open
MxmUrw wants to merge 8 commits intomainfrom
chore/remove-assethub-from-bouncer
Open

chore: remove assethub from bouncer#6435
MxmUrw wants to merge 8 commits intomainfrom
chore/remove-assethub-from-bouncer

Conversation

@MxmUrw
Copy link
Contributor

@MxmUrw MxmUrw commented Feb 23, 2026

Pull Request

Closes: PRO-2755

Summary

This PR removes Assethub tests from bouncer. Code that was immediately used by the tests has been removed, but the assets still contain assethub of course. Also bouncer commands for assethub (send dot, get balance, ...) have been kept for now.


Checklist [feel free to add/remove/edit as appropriate but please don't ignore]

Before marking this as ready for review, consider the following:

  • The PR is complete:
    • Scope is clear and fully implemented.
  • Make life easy for the reviewer:
    • Intended behaviours are clear and, where possible, covered by tests.
    • Unrelated changes are isolated in dedicated commits.
    • Anything non-obvious is documented in the Summary section above.

@codecov
Copy link

codecov bot commented Feb 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@MxmUrw MxmUrw force-pushed the chore/remove-assethub-from-bouncer branch from 9d8ea65 to 7a63b08 Compare February 23, 2026 14:30
@MxmUrw MxmUrw force-pushed the chore/remove-assethub-from-bouncer branch from 7a63b08 to da2f457 Compare February 23, 2026 14:33
@MxmUrw MxmUrw changed the title chore: remove assethub from bouncer chore: [don't merge yet] remove assethub from bouncer Feb 23, 2026
@MxmUrw MxmUrw marked this pull request as ready for review February 23, 2026 15:18
@MxmUrw MxmUrw requested a review from dandanlen February 23, 2026 15:18
@MxmUrw MxmUrw changed the title chore: [don't merge yet] remove assethub from bouncer chore: remove assethub from bouncer Feb 27, 2026
@MxmUrw
Copy link
Contributor Author

MxmUrw commented Feb 27, 2026

Since release/2.1 is branched, this can be reviewed and merged now. Though probably after the bouncer refactor.

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.

2 participants