default of `runs: 1` overrides a config's numberOfRuns

[treosh]

default of `runs: 1` overrides a config's numberOfRuns

[alekseykulikov]

So we shouldn't set the CLI numberOfRuns if it was just falling back to the actions' default of 1.

I agree with this strategy for now.

the action could just default to the default of the CLI (which is 3 runs)

I don't think that the 3-runs is an optimal default. It may results in a 3x bill with no significant value. 1-run works for static checks, like resource budgets or lighthouse scores. 3-runs makes sense when user tests performance or collect analytics with LHCI Server.

Maybe we could fallback to LHCI default (breaking change for LH v6 release) and show a warning when 3-runs used without an extra value.

[alekseykulikov]

fixed in treosh/lighthouse-ci-action@v3