Skip to content

Conversation

@ilgooz
Copy link
Member

@ilgooz ilgooz commented Jul 24, 2022

No description provided.

ilgooz and others added 13 commits June 23, 2022 20:10
chore: merge master
Co-authored-by: jeronimoalbi <jeronimo.albi@tendermint.com>
* fix(pkg/cosmosanalysis): support App with generics

* test: turn into parametized test and use embed for file content

Co-authored-by: jeronimoalbi <jeronimo.albi@tendermint.com>
* docs: replace docker deprecated cmds

Also:
- adds a preliminary step related to the `$HOME/sdh` creation and a
  rationale.
- remove `-w /app` flag since the default is `/apps` and it works well
  with it.
- fix docker hub URL which was changed to `ignite/cli` instead of
  `ignitehq/cli`
- use public docker hub URL (previous version was private and redirects
  to the login page of docker hub)

Fix #2596

* Update docs/docs/kb/09-docker.md

Co-authored-by: Jerónimo Albi <jeronimo.albi@tendermint.com>

* Update docs/docs/kb/09-docker.md

Co-authored-by: Jerónimo Albi <jeronimo.albi@tendermint.com>

Co-authored-by: Jerónimo Albi <jeronimo.albi@tendermint.com>
Co-authored-by: Thomas Bruyelle <thomas.bruyelle@gmail.com>
The change fixes a go command error when scaffolding a chain (see #2618), because go cannot fetch `ignite-hq/cli` despite the github redirection.

To remove `ignite-hq/cli` from the dependency list, I had to : 
- upgrade `spn` to latest (so it depends on `ignite/cli@v0.22.2` rather than `ignite-hq/cli`)
- upgrade go to 1.18 in `go.mod.plush` (or else `ignite/cli@v0.22.2` adds a dependency to an older `spn` which in turn adds a dependency to `ignite-hq/cli`)

Bonus:
- improve `go mod download` error report : add the binary output in addition to `Exit status 1`.
- fix test because of upgrade to ibc-go 3.0.1, where  some parameters can't be null.
* fix: migration docs for v0.22.0

* docs: add migration docs for v0.22.2

* fix: changed docs for migration steps

* fix: migration docs for the `main.go` file in the `cmd/` folder

* fix: use suggested comment for item 6

* fix: change migration guide to recommend bulk find and replace
* docs: change install version in guide

* change: fix blog guide documentation

* change: fix guide documentation for blogclient

* change: fix guide documentation for DeFi loan module

* change: fix guide documentation for IBC

* change: fix comment delete in blog guide documentation

* change: fix guide documentation for nameservice

* change: fix guide documentation for scavenge

* docs: add spacing to code snippets in guides for readability

* change: fix guide documentation for interchange

* docs(guide): fix submit scavenge address source

Co-authored-by: Thomas Bruyelle <thomas.bruyelle@tendermint.com>
Co-authored-by: Alex Johnson <alex.johnson@tendermint.com>
Co-authored-by: İlker G. Öztürk <ilkergoktugozturk@gmail.com>
* fix(pkg/cosmosgen): IBC proto messages import

* refactor: change cosmosgen third party proto include paths resolution

* fix(pkg/cosmosgen): add global protoc includes to generator

* chore: remove unused protopath package

Co-authored-by: İlker G. Öztürk <ilkergoktugozturk@gmail.com>
* docs: add v0.22.3 section to changelog

* docs: add newly merged PRs to v0.22.3 section in changelog
@ilgooz ilgooz merged commit 64df9ae into master Jul 24, 2022
@github-actions
Copy link
Contributor

Visit the preview URL for this PR (updated for commit a6886e2):

https://ignite-go-docs--pr2652-develop-6i6y6bh9.web.app

(expires Sun, 31 Jul 2022 18:15:57 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Jchicode pushed a commit to Jchicode/cli that referenced this pull request Aug 9, 2023
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.

4 participants