Re: erl_tidy option to print to stdout
Andrew Thompson <[email protected]>
| Newsgroups | gmane.comp.lang.erlang.patches |
|---|---|
| Message-ID | <[email protected]> |
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 _______________________________________________ erlang-patches mailing list [email protected] http://erlang.org/mailman/listinfo/erlang-patches