search

jarhart / SublimeSBT

SBT build tool integration for Sublime Text 2 and Sublime Text 3.
MIT License
98 stars 10 forks source link

Cause the filter object to be evaluated instead of returning the object #6

Closed coltfred closed 11 years ago

coltfred commented 11 years ago

The change you made on line 84 actually breaks functionality and any boolean expression written against the function to return true. This is caused by the fact that the filter object itself is returned. What you want is the following:

list(filter(self._is_play_build, self._scala_build_files(folder)))

I've fixed it in this branch.

jarhart commented 11 years ago

You are correct, sir! I neglected to test this fully! Your fix is good too, as it has that method return a list, consistent with its siblings.

Thanks!