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

feat: remove orchestrator address #1330

Merged
merged 2 commits into from
Feb 3, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Feb 2, 2023

Overview

Closes #1329

Depends on a new release of the cosmos-sdk.

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id added the x/qgb label Feb 2, 2023
@rach-id rach-id self-assigned this Feb 2, 2023
@MSevey MSevey requested a review from a team February 2, 2023 15:09
go.mod Outdated
@@ -185,7 +185,7 @@ require (

replace (
github.com/celestiaorg/nmt v0.13.0 => github.com/celestiaorg/nmt v0.12.0
github.com/cosmos/cosmos-sdk => github.com/celestiaorg/cosmos-sdk v1.7.0-sdk-v0.46.7
github.com/cosmos/cosmos-sdk => github.com/celestiaorg/cosmos-sdk v1.7.0-sdk-v0.46.7.0.20230202145553-0b25d300d771
Copy link
Member Author

Choose a reason for hiding this comment

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

[note for reviewers]
will be changed by the new cosmos-sdk release

Copy link
Member

@evan-forbes evan-forbes left a comment

Choose a reason for hiding this comment

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

noice, let's bump the sdk and :shipit:

@Bidon15
Copy link
Member

Bidon15 commented Feb 3, 2023

Awesome! I wonder to which release this is planned to be included?
As test-infra also needs adaptations on removing orchestrator address

rahulghangas
rahulghangas previously approved these changes Feb 3, 2023
@evan-forbes
Copy link
Member

@sweexordious we just bumbed to v1.8.0-sdk-0.46.7, mind updating here? we can also do in a follow up PR

@evan-forbes evan-forbes merged commit f0936da into celestiaorg:main Feb 3, 2023
evan-forbes pushed a commit that referenced this pull request Feb 27, 2023
<!--
Please read and fill out this form before submitting your PR.

Please make sure you have reviewed our contributors guide before
submitting your
first PR.
-->

## Overview

<!-- 
Please provide an explanation of the PR, including the appropriate
context,
background, goal, and rationale. If there is an issue with this
information,
please provide a tl;dr and link the issue. 
-->

Closes #1329

Depends on a new release of  the cosmos-sdk.

## Checklist

<!-- 
Please complete the checklist to ensure that the PR is ready to be
reviewed.

IMPORTANT:
PRs should be left in Draft until the below checklist is completed.
-->

- [ ] New and updated code has appropriate documentation
- [ ] New and updated code has new and/or updated testing
- [ ] Required CI checks are passing
- [ ] Visual proof for any user facing features like CLI or
documentation updates
- [ ] Linked issues closed with keywords
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.

QGB remove orch address
4 participants