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.

Sign up for GitHub

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

Jump to bottom

Fix CI #8

Closed
wants to merge 5 commits into from
Closed

Fix CI #8

wants to merge 5 commits into from

Conversation

yito88
Copy link
Member

@yito88 yito88 commented Oct 11, 2023 •
edited
Loading

  • Install protoc for CI
  • Workaround the config write failure
    • Namada byte-unit requires u128
    • Added Byte(u64)
  • Add E2E test with Namada

@yito88 yito88 force-pushed the yuji/fix-ci branch 2 times, most recently from 3b98499 to 553c097 Compare October 21, 2023 13:18
@yito88 yito88 marked this pull request as ready for review November 1, 2023 10:16
@yito88 yito88 requested a review from sug0 November 1, 2023 10:16
@yito88
Copy link
Member Author

yito88 commented Nov 10, 2023

We will update Hermes based on 1.7.0 with new Namada. These changes for CI aren't needed anymore.

@yito88 yito88 closed this Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sug0 sug0 Awaiting requested review from sug0

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@yito88