[erlang-patches] erl_tidy option to print to stdout

Andrew Thompson andrew@REDACTED
Thu Jul 11 18:54:29 CEST 2013


On Thu, Jul 11, 2013 at 04:48:12PM +0200, Björn-Egil Dahlberg wrote:
> It is great that the new option is documented!
> 
> Especially since syntax_tools is lacking tests, I think it's crucial
> that tests are added.
> 
> It is our policy to require tests for all new features unless the
> feature is already covered under some other test.

Not that I don't applaud this policy, but it can be *extremely*
difficult to add new tests to a untested/minimally tested codebase.

I know when I've made contributions in the past I've tried to update the
test suites where applicable, but when there's no tests I've sort of
thrown up my hands and just submitted the patch.

I don't know the best solution to this, but pushing all the work on the
contributor seems a little unfair. Maybe the OTP team can help out with
testing in such cases.

Andrew



More information about the erlang-patches mailing list