> So, removing all the options makes it harder for tests to consume > this, doesn't it? Is there any reason not to keep them?
The config file is yaml, so it's very easy to produce. The difference in effort between that and composing a long command line is very small.
Tests can and should use txlongpollfixture.
It's work to maintain both, work which I - none of us - have time for.
We should never pass in credentials via the command-line, so we need a config file for that at least. Why not just go the whole way?
This is going to be in main for Precise. We might upset the security team if we permit credentials in the command-line.
« Back to merge proposal
> So, removing all the options makes it harder for tests to consume
> this, doesn't it? Is there any reason not to keep them?
The config file is yaml, so it's very easy to produce. The difference in effort between that and composing a long command line is very small.
Tests can and should use txlongpollfixture.
It's work to maintain both, work which I - none of us - have time for.
We should never pass in credentials via the command-line, so we need a config file for that at least. Why not just go the whole way?
This is going to be in main for Precise. We might upset the security team if we permit credentials in the command-line.