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

docs: adding details of why the ADR 012 was rejected #1430

Merged
merged 1 commit into from
Nov 15, 2023
Merged

docs: adding details of why the ADR 012 was rejected #1430

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion docs/docs/adrs/adr-012-separate-releasing.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,11 @@ We upgrade `main` to use a new version of SDK. This is a major version bump, tri
### Negative

* ~~Slightly more complexity.~~Considerably more complex to manage the ICS library.
* This solution does not allow having provider and consumer on separate versions of e.g. the Cosmos SDK
This is because ICS needs to support multiple versions of SDK (e.g., 0.45, 0.47, 0.50).
In addition, ICS needs to support a special fork of SDK (with LSM included) for the Cosmos Hub.
This means that instead of focusing on main the development team needs to manage multiple release
branches with different dependency trees.
* This solution does not allow having provider and consumer on separate versions of e.g. the Cosmos SDK.

### Neutral

Expand Down