Skip to content

Comments

migrate answers example from yaml to json#1858

Merged
jreidinger merged 2 commits intomasterfrom
update_answers_example
Jan 15, 2025
Merged

migrate answers example from yaml to json#1858
jreidinger merged 2 commits intomasterfrom
update_answers_example

Conversation

@jreidinger
Copy link
Contributor

Problem

When changing answers format from yaml to json example file was not converted.

Solution

Convert it and also use example which provides password.

Copy link
Contributor

@mvidner mvidner left a comment

Choose a reason for hiding this comment

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

May I suggest a reformatted version (via python -m json.tool)

{
    "answers": [
        {
            "answer": "decrypt",
            "class": "storage.luks_activation",
            "password": "my_password"
        }
    ]
}

Copy link
Contributor

@mvidner mvidner left a comment

Choose a reason for hiding this comment

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

Still LGTM :)

@jreidinger jreidinger merged commit e3944ce into master Jan 15, 2025
4 checks passed
@jreidinger jreidinger deleted the update_answers_example branch January 15, 2025 13:50
@imobachgs imobachgs mentioned this pull request Feb 26, 2025
imobachgs added a commit that referenced this pull request Feb 26, 2025
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.

2 participants