← Back to team overview

mvhub-dev team mailing list archive

Re: [Merge] lp:~pulaparti/mvhub/fix_html_lint into lp:mvhub

 

Review: Approve
looks good enough, ok to merge

Slightly better would be:

@files = grep { ! |conf/templates/text| } @files;

or:

@files = grep { ! #conf/templates/text# } @files;

You can have different delimiters for regular expressions, not using / means you don't have to escape them and it is a little easier to read.
-- 
https://code.launchpad.net/~pulaparti/mvhub/fix_html_lint/+merge/21969
Your team mvhub-dev is subscribed to branch lp:mvhub.



References