Optional inputs don't work
See original GitHub issueIf I run the deploy command with optional input values, I get always the error Too many arguments, expected arguments "command" "stage".
If I run this command from my local machine, everything works fine.
This is my configuration
- name: Deploy website
uses: deployphp/action@v1.0.5
with:
private-key: ${{ secrets.SSH_PRIVATE_KEY }}
known-hosts: ${{ secrets.SSH_KNOWN_HOSTS }}
dep: deploy ${{ steps.extract-branch.outputs.branch }} --composer_auth '${{ env.composer_auth }}'
And this is in my PHP file:
use Symfony\Component\Console\Input\InputOption;
option('composer_auth', null, InputOption::VALUE_OPTIONAL, 'Add a composer authentification configuration');
Issue Analytics
- State:
- Created 3 years ago
- Comments:14 (7 by maintainers)
Top Results From Across the Web
Why should Java 8's Optional not be used in arguments
Usually it is not the case. If not, I use Optional as argument types and/or field types. There are still grey areas, where...
Read more >Optional parameters don't work · Issue #122 - GitHub
If I add constraint to route pattern for method ("[controller]/{id:int?}") it strats work again but only with supplied id value.
Read more >passing optional parameters to controllers doesn't work
4 Bug(?): passing optional parameters to controllers doesn't work. Can anyone please verify this finding, which I consider a bug in Laravel 5.4:...
Read more >Optional Input does not work - Grasshopper Developer
pManager[2].Optional = true;. However, my componenet still doesnt run. It gives me this error. Input parameter failed to collect data.
Read more >Using Python Optional Arguments When Defining Functions
In this tutorial, you'll learn about Python optional arguments and how to define functions with default values. You'll also learn how to create...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Yes, looks like there can be a better solution with correct splitting in js code. Will implement it.
Yeah, it works now as expected, thank you a lot 🎉