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: add flags to the extension hooks commands #4270

Merged
merged 8 commits into from
Jul 25, 2024
Merged

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Jul 19, 2024

close ignite/apps#118

Description

This PR adds the ability to extend the command flags using extension hooks and also passes the flags to the hook interface to be accessible into the extension.

@Pantani Pantani self-assigned this Jul 19, 2024
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs component:cmd type:services Service-related issues. labels Jul 19, 2024
@Pantani Pantani changed the title feat: add flags to the hook commands feat: add flags to the extestion hooks commands Jul 19, 2024
@Pantani Pantani requested review from salmad3 and toschdev as code owners July 19, 2024 12:37
@Pantani Pantani added the backport/v28.x.y Backport to v28.x.y label Jul 19, 2024
@julienrbrt julienrbrt changed the title feat: add flags to the extestion hooks commands feat: add flags to the extension hooks commands Jul 22, 2024
julienrbrt
julienrbrt previously approved these changes Jul 22, 2024
Copy link
Member

@julienrbrt julienrbrt left a comment

Choose a reason for hiding this comment

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

utACK!

ignite/cmd/plugin.go Show resolved Hide resolved
changelog.md Outdated Show resolved Hide resolved
@Pantani Pantani requested a review from julienrbrt July 25, 2024 20:41
@Pantani Pantani enabled auto-merge (squash) July 25, 2024 21:10
@Pantani Pantani merged commit 54e7f52 into main Jul 25, 2024
45 checks passed
@Pantani Pantani deleted the feat/ext-hooks-flags branch July 25, 2024 21:21
mergify bot pushed a commit that referenced this pull request Jul 25, 2024
* add flags to the hook commands

* add flags to executed hook cmd

* create hook.ImportFlags method

* add changelog

* Update changelog.md

(cherry picked from commit 54e7f52)

# Conflicts:
#	ignite/services/plugin/grpc/v1/interface.pb.go
julienrbrt added a commit that referenced this pull request Jul 29, 2024
* feat: add flags to the extension hooks commands (#4270)

* add flags to the hook commands

* add flags to executed hook cmd

* create hook.ImportFlags method

* add changelog

* Update changelog.md

(cherry picked from commit 54e7f52)

# Conflicts:
#	ignite/services/plugin/grpc/v1/interface.pb.go

* `make proto-gen`

* updates

---------

Co-authored-by: Danilo Pantani <[email protected]>
Co-authored-by: Julien Robert <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/v28.x.y Backport to v28.x.y component:ci CI/CD workflow and automated jobs. component:cmd component:configs type:services Service-related issues.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

INIT: Enable Extension to Extend Flag in Ignite Command Using Hook
2 participants