|
From: Luigi B. <lui...@gm...> - 2015-12-10 08:02:54
|
A once-off total reformatting is probably not worth your time :) Ok, send me a PR. Luigi On Thu, Dec 10, 2015 at 7:30 AM Francois Botha <ig...@gm...> wrote: > I think it's up to the specific plugin of whatever IDE one uses on how the > configuration is applied. But I agree with you. One doesn't want to see > non-trivial diffs in the commits. I would think that a user who has the > knowledge to consume these format configuration files would also know to > remove irrelevant changes when doing a commit? > > And I assume a once-off total reformatting of the project is going to > create too many conflicts on all the branches and forks? I'm not too > concerned with how the curly braces are positioned, but consistent > indentation and trailing whitespace removal would be nice for a start. > > Francois > > Francois Botha > > On 9 December 2015 at 17:04, Luigi Ballabio <lui...@gm...> > wrote: > >> No particular objection. I'd just try to avoid having cosmetic changes >> making it more difficult to see what actually changed in a commit, so I'd >> suggest to configure editors so that they only apply the conventions to >> edited lines, not over the whole file. Is this possible? >> >> Luigi >> >> On Wed, Nov 25, 2015 at 10:18 AM Francois Botha <ig...@gm...> wrote: >> >>> Hi, >>> >>> I was wondering whether there are any objections against adding >>> .editorconfig files to all the projects. This will assist in maintaining >>> consistent formatting (especially indentation). Most IDEs are supported. >>> See http://editorconfig.org/ for more info. >>> >>> regards, >>> Francois Botha >>> >>> ------------------------------------------------------------------------------ >>> Go from Idea to Many App Stores Faster with Intel(R) XDK >>> Give your users amazing mobile app experiences with Intel(R) XDK. >>> Use one codebase in this all-in-one HTML5 development environment. >>> Design, debug & build mobile apps & 2D/3D high-impact games for multiple >>> OSs. >>> http://pubads.g.doubleclick.net/gampad/clk?id=254741551&iu=/4140 >>> _______________________________________________ >>> QuantLib-dev mailing list >>> Qua...@li... >>> https://lists.sourceforge.net/lists/listinfo/quantlib-dev >>> >> -- >> >> <http://leanpub.com/implementingquantlib> >> <http://implementingquantlib.com> >> <http://twitter.com/lballabio> >> > > -- <http://leanpub.com/implementingquantlib> <http://implementingquantlib.com> <http://twitter.com/lballabio> |