Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor: add icq relayer to the quicksilver repository #553

Closed

Conversation

faddat
Copy link
Contributor

@faddat faddat commented Sep 11, 2023

1. Summary

Add the icq relayer to the qucksilver repository

2.Type of change

Monorepo refactor

3. Implementation details

Just moved the code in here from elsewhere.

4. How to test/use

cd icq-relayer
go test ./...

5. Checklist

@codecov
Copy link

codecov bot commented Sep 14, 2023

Codecov Report

Merging #553 (846b9df) into develop (a28e49a) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #553   +/-   ##
========================================
  Coverage    13.19%   13.19%           
========================================
  Files          243      243           
  Lines        62942    62942           
========================================
  Hits          8306     8306           
  Misses       53967    53967           
  Partials       669      669           
Flag Coverage Δ
unittests 13.19% <ø> (ø)

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

@joe-bowman
Copy link
Contributor

Ser, we need the icq-relayer tagged v0.9.1 which is the latest, rather than main.

@faddat
Copy link
Contributor Author

faddat commented Sep 15, 2023

Gotcha. I'll do that.

@faddat faddat requested a review from ThanhNhann as a code owner September 15, 2023 05:32
Copy link
Contributor

@joe-bowman joe-bowman left a comment

Choose a reason for hiding this comment

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

wrong repo ;)

@joe-bowman joe-bowman closed this Sep 15, 2023
@joe-bowman
Copy link
Contributor

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