You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Perhaps eventually a new major release where we change how all the parameters
are passed could sort it out so that instead we are explicit about the tuples
of directory+tfversion+args instead of trying to match them across three
arrays.
Combining the three parameters into tuples sounds great for more complex cases, but I'm worried that the change will make this action looks too complicated for people using this with simpler cases (e.g checking just one directory with the latest version of Terraform)
Instead of replacing the current input parameters, perhaps we can introduce a new parameter for advanced usage while keeping the existing parameters? This way we don't need to introduce breaking changes either. I'm thinking about something like this (haven't really thought about the naming or whether this is valid):
Combining the three parameters into tuples sounds great for more complex cases, but I'm worried that the change will make this action looks too complicated for people using this with simpler cases (e.g checking just one directory with the latest version of Terraform)
Instead of replacing the current input parameters, perhaps we can introduce a new parameter for advanced usage while keeping the existing parameters? This way we don't need to introduce breaking changes either. I'm thinking about something like this (haven't really thought about the naming or whether this is valid):
Originally posted by @hanscg in #23 (comment)
The text was updated successfully, but these errors were encountered: