perltidyrc with minimal amount of changes to code

ingydotnet / io-all-pm

All in One Perl IO

http://search.cpan.org/dist/IO-All/

38 stars 20 forks source link

perltidyrc with minimal amount of changes to code #59

Open wchristian opened 9 years ago

wchristian commented 9 years ago

Based on the recent irc discussion i made a .perltidyrc file which results in the minimum amount of changes i could configure it for.

The bulk of the changes it does is vertical alignment in lines with similar content, and indenting of qw statements. As you go through you'll however also find that it improves some parts of the code to make them more consistent.

Might be worth taking this in, as a one-time thing and asking people who make patches to tidy the code with the included perltidyrc in the future.

If you don't like it, no worry, i won't cry. :)

frioux commented 9 years ago

I hate that perltidy can't be configured to not align things like that. It just causes a ton of churn.

wchristian commented 9 years ago

Everyone does, but i guess the situation is as always "patches welcome". :)

shlomif commented 7 years ago

Sorry for the late reply, but I'm getting many conflicts when trying to merge it and my git skills are not up-to-the-task. Can you redo this pull req?

wchristian commented 7 years ago

Redid it and force-pushed it, please fetch and reset to f7bbd93e892dddd9b269a9b1ce2bb23dc9db2bf7

coveralls commented 7 years ago

Coverage increased (+0.7%) to 90.769% when pulling f7bbd93e892dddd9b269a9b1ce2bb23dc9db2bf7 on wchristian:perltidyrc into 042a1515f7b8db0792e2d74777806fc4ff2a2d92 on ingydotnet:master.