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

support a multi-* declaration (should support params, outputs at least, but likely more) #23

Open
alex-frankel opened this issue Jun 8, 2020 · 3 comments
Labels
syntax Related to language syntax

Comments

@alex-frankel
Copy link
Collaborator

Parameters probably should always be declared at the top of the file anyway, and this should help with readability and terse-ness:

parameters {
   prefix string = 'my string'
   suffix string {
    secure: 'true'
  }
  ipRange object {
    defaultValue: {
      myProp: 'my value'
    }
  }
}

would be nice if I can separate over multiple-lines:

parameters {
   prefix string = 'my string'
   
   suffix string {
    secure: 'true'
  }
  
   ipRange object {
    defaultValue: {
      myProp: 'my value'
    }
  }
}
@majastrz majastrz added the syntax Related to language syntax label Jun 9, 2020
@majastrz
Copy link
Member

majastrz commented Jun 9, 2020

Would we allow the user to mix single-parameter declarations with this one?

@alex-frankel
Copy link
Collaborator Author

good question - it might be an interesting way to enforce the behavior that all parameters should be declared together

@majastrz
Copy link
Member

Meeting notes from 6/10/2020:

  • Syntax looks too much like JSON without being an actual JSON object [@filizt and @snarkywolverine]
  • Should require parameters at the top of the file [@marcre]

@majastrz majastrz mentioned this issue Jul 1, 2020
@alex-frankel alex-frankel changed the title support a multi-parameter declaration support a multi-* declaration (should support params, outputs at least, but likely more) Aug 13, 2020
@alex-frankel alex-frankel added this to the v0.3 milestone Aug 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
syntax Related to language syntax
Projects
None yet
Development

No branches or pull requests

2 participants