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.

Sign up for GitHub

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

Jump to bottom

fix: Improve bind imports, support lower case 'source' in yaml, add go test #517

Merged
merged 1 commit into from
Oct 3, 2023
Merged

fix: Improve bind imports, support lower case 'source' in yaml, add go test #517

merged 1 commit into from
Oct 3, 2023

Commits on Oct 3, 2023

  1. fix: Improve bind imports, support lower case 'source' in yaml, add g… 

    …o test

Entries stacker.yaml's "binds" are of type Bind.
When imported from yaml, support was present for these three things:
1. ascii art specifying source and dest ("source -> dest")
2. source only ("source")
3. map with 'Source' and 'Dest'.

Support for '3' is dropped, and replaced by map with 'source' and
'dest'.  'Source' and 'Dest' are inconsistent with the rest of stacker.yaml.
Note that importing from json still does support capital letter variants
as the golang json library is case insensitive.

Improvements to the bind yaml and json support generally rely more
heavily on the library to do the lifting.

Signed-off-by: Scott Moser <smoser@brickies.net>
    @smoser
    smoser committed Oct 3, 2023
    Configuration menu
    Copy the full SHA
    fe83356 View commit details
    Browse the repository at this point in the history