Skip to content

Conversation

@aljo242
Copy link
Contributor

@aljo242 aljo242 commented Nov 1, 2022

  • Remove code for old SDK versions (Pre-stargate).
  • simplifies cosmosver pkg by removing the Family field.

@aljo242 aljo242 added the type:new To implement new feature. label Nov 1, 2022
@aljo242 aljo242 self-assigned this Nov 1, 2022
tbruyelle
tbruyelle previously approved these changes Nov 2, 2022
Copy link
Contributor

@lumtis lumtis left a comment

Choose a reason for hiding this comment

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

We have the Plugin interface in services/chain that could be removed unless we consider it as future proof in case new version with different API come up, but it seems unlikely

It complexifies the codebase right now

@aljo242
Copy link
Contributor Author

aljo242 commented Nov 3, 2022

We have the Plugin interface in services/chain that could be removed unless we consider it as future proof in case new version with different API come up, but it seems unlikely

It complexifies the codebase right now

We should at least modify the name from plugin which is very unclear at this point. Opened #3049

@lumtis
Copy link
Contributor

lumtis commented Nov 3, 2022

Great, let's merge this one

@lumtis lumtis merged commit f239032 into develop Nov 3, 2022
@lumtis lumtis deleted the refactor/remove-launchpad branch November 3, 2022 16:29
Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
* remove family

* remove family

* remove StarportOnlySupported error

* remove dead code

* lint fixes

* changelog

Co-authored-by: Lucas Bertrand <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:new To implement new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants