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

feat: allow customization of trusting_period #76

Merged
merged 7 commits into from
Jul 26, 2024
Merged

feat: allow customization of trusting_period #76

merged 7 commits into from
Jul 26, 2024

Conversation

gjermundgaraba
Copy link
Contributor

closes #70

@gjermundgaraba gjermundgaraba requested a review from srdtrk as a code owner July 24, 2024 11:23
@gjermundgaraba gjermundgaraba linked an issue Jul 24, 2024 that may be closed by this pull request
Allow customization of trusting_period #70
Closed
srdtrk
srdtrk approved these changes Jul 25, 2024
View reviewed changes
Copy link
Member

@srdtrk srdtrk left a comment

Choose a reason for hiding this comment

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

Looks good to me! Thanks. I left a couple questions. Can merge this once we resolve those

operator/src/runners/fixtures/membership.rs Show resolved Hide resolved
operator/src/cli/command.rs Outdated Show resolved Hide resolved
justfile Outdated Show resolved Hide resolved
e2e/interchaintestv8/sp1_ics07_test.go Outdated Show resolved Hide resolved
@gjermundgaraba gjermundgaraba requested a review from srdtrk July 26, 2024 09:44
@srdtrk srdtrk merged commit 1dffbb0 into main Jul 26, 2024
12 checks passed
@srdtrk srdtrk deleted the gjermund/70-allow-customization-of-trusting_period branch July 26, 2024 21:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@srdtrk srdtrk Awaiting requested review from srdtrk srdtrk is a code owner

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

Successfully merging this pull request may close these issues.

Allow customization of trusting_period
2 participants
@gjermundgaraba @srdtrk