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/load config interface #544

Merged
merged 17 commits into from
Dec 23, 2024
Merged

Feat/load config interface #544

merged 17 commits into from
Dec 23, 2024

Conversation

ldmonster
Copy link
Contributor

@ldmonster ldmonster commented Dec 13, 2024

Overview

Add interface between module/globalConfig and hooks to get options.

What this PR does / why we need it

Incapsulate options logic in hook

Special notes for your reviewer

Pavel Okhlopkov added 3 commits December 13, 2024 19:48
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
@ldmonster ldmonster force-pushed the feat/load-config-interface branch from 46282ee to 06298c8 Compare December 14, 2024 12:54
@ldmonster ldmonster added the enhancement New feature or request label Dec 16, 2024
Pavel Okhlopkov added 2 commits December 16, 2024 11:24
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
@ldmonster ldmonster requested a review from yalosev December 16, 2024 08:35
@ldmonster ldmonster marked this pull request as ready for review December 16, 2024 08:35
Pavel Okhlopkov added 2 commits December 16, 2024 12:49
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
@ldmonster ldmonster requested a review from miklezzzz December 18, 2024 10:58
@ldmonster ldmonster self-assigned this Dec 19, 2024
Pavel Okhlopkov added 9 commits December 19, 2024 21:29
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
Signed-off-by: Pavel Okhlopkov <pavel.okhlopkov@flant.com>
@yalosev yalosev merged commit 8ab6d62 into main Dec 23, 2024
7 checks passed
@yalosev yalosev deleted the feat/load-config-interface branch December 23, 2024 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants