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

Load pipeline on yaml tree vs go structs #468

Closed
wants to merge 35 commits into from

Conversation

ndeloof
Copy link
Collaborator

@ndeloof ndeloof commented Oct 9, 2023

This replaces use of mergo and various reflection hacks with a plain yaml tree (map[string]any) processing

Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof force-pushed the pure_yaml_override branch 5 times, most recently from 9b8be2a to 00f65e8 Compare October 10, 2023 08:40
@ndeloof ndeloof force-pushed the pure_yaml_override branch from 00f65e8 to 12e2ddd Compare October 10, 2023 08:46
Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof force-pushed the pure_yaml_override branch from b3785d0 to 0da7863 Compare October 12, 2023 13:59
Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof force-pushed the pure_yaml_override branch from 3e601b8 to 14595c9 Compare October 13, 2023 09:46
Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof force-pushed the pure_yaml_override branch from 14595c9 to 73eb7f3 Compare October 13, 2023 09:47
Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof marked this pull request as ready for review October 17, 2023 08:25
Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof ndeloof force-pushed the pure_yaml_override branch from 24fb874 to 5f6ee40 Compare October 17, 2023 09:27
Signed-off-by: Nicolas De Loof <[email protected]>
Signed-off-by: Nicolas De Loof <[email protected]>
@ndeloof
Copy link
Collaborator Author

ndeloof commented Oct 20, 2023

@ndeloof ndeloof closed this Oct 20, 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.

1 participant