Skip to content

feat: set the proto-dir flag only for the scaffold chain command and use the proto path from the config - #4100

Merged
julienrbrt merged 12 commits into
mainfrom
feat/proto-dir-from-config
Apr 25, 2024
Merged

feat: set the proto-dir flag only for the scaffold chain command and use the proto path from the config#4100
julienrbrt merged 12 commits into
mainfrom
feat/proto-dir-from-config

Conversation

@Pantani

@Pantani Pantani commented Apr 23, 2024

Copy link
Copy Markdown
Contributor

Description

  • Only use the proto-dir flag for scaffold chain command;
  • Only use the proto path from the config since the generate commands are always running and getting from the config file;

@Pantani Pantani self-assigned this Apr 23, 2024
@github-actions github-actions Bot added component:ci CI/CD workflow and automated jobs. component:configs component:cmd labels Apr 23, 2024
@Pantani
Pantani marked this pull request as draft April 23, 2024 14:51
@Pantani Pantani changed the title feat: only use the proto dir from config feat: set the proto-dir flag only for the scaffold chain command and use the proto path from the config Apr 24, 2024
@Pantani
Pantani marked this pull request as ready for review April 24, 2024 00:43
Comment thread integration/app/cmd_proto_path_test.go Outdated
Comment thread ignite/cmd/cmd.go
@Pantani
Pantani requested a review from julienrbrt April 24, 2024 23:13
@julienrbrt
julienrbrt merged commit af41d3d into main Apr 25, 2024
@julienrbrt
julienrbrt deleted the feat/proto-dir-from-config branch April 25, 2024 08:59
julienrbrt pushed a commit that referenced this pull request May 29, 2024
…and use the proto path from the config (#4100)

* only use the proto dir from config

* add proto dir only for scaffold a new chain

* add changelog

* fix integartion tests

* remove unused flag from integration tests cmds

* fix revive lint use

* add a helper to find the root chain path

* remove unused method

* remove unused tests

---------

Co-authored-by: Pantani <Pantani>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants