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

poor error message: more than one mixin specified #370

Closed
carolynvs opened this issue May 31, 2019 · 0 comments · Fixed by #3052
Closed

poor error message: more than one mixin specified #370

carolynvs opened this issue May 31, 2019 · 0 comments · Fixed by #3052
Labels
user experience 🌈💖 Make it easier for everyone to use Porter

Comments

@carolynvs
Copy link
Member

carolynvs commented May 31, 2019

When the yaml for an action step is malformed, and more than one key is at the root of the step, you trigger "more than one mixin specified". But the error message isn't clear on what is wrong and how to fix it.

Let's help people out by:

  • Improving the error message
  • Adding the error message to the FAQ and explain how to fix

Reproduction Steps

  1. make a new directory and inside it run porter create
  2. Edit porter.yaml and mess up the indentation of the install step (note that description and the other fields should be indented underneath exec)
    install:
        - exec:
          description: "Install Hello World"
          command: ./helpers.sh
          arguments:
            - install
  3. Run porter build
$  porter build
Using buildkit build driver
1 error occurred:
	* validation of action "install" failed: more than one mixin specified

Ideally the error message should say that the first step in the install action is malformed and they should check that all of the mixins's child fields are indented underneath the name of the mixin.

@carolynvs carolynvs added the user experience 🌈💖 Make it easier for everyone to use Porter label May 31, 2019
@carolynvs carolynvs added the needs-followup Needs a maintainer to follow-up label Sep 29, 2022
@carolynvs carolynvs removed the needs-followup Needs a maintainer to follow-up label Sep 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
user experience 🌈💖 Make it easier for everyone to use Porter
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant