mvhub-dev team mailing list archive
-
mvhub-dev team
-
Mailing list archive
-
Message #00010
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