byuitechops / action-series-master

The master module for all of the action-series child modules.
3 stars 0 forks source link

Switch to regex.text() #23

Closed zachintosh closed 6 years ago

zachintosh commented 6 years ago

https://github.com/byuitechops/action-series-master/blob/6242a2ffc0f996c03860c1fecf92d3674e47ac5e/actions/universal-remove-banners.js#L10

Using regex.match() returns an array of all matches in the text. Give regex.test() a shot instead. It just returns true or false, which is what the if statement needs.

csammcgrath commented 6 years ago

Should be fixed with the latest pull request.