def on_pr_opened(self, api, payload):
diff = api.get_diff()
for line in diff.split('\n'):
if is_addition(line) and line.find("<title></title>") > -1:
# This test doesn't consider case and whitespace the same way
# that a HTML parser does, so empty title elements might still
# go unnoticed. It will catch the low-hanging fruit, though.
self.warn(WARNING)
return
def get_added_lines(self):
diff = self.get_diff()
for line in diff.splitlines():
if is_addition(line):
# prefix of one or two pluses (+)
yield line
def on_pr_opened(self, api, payload):
diff = api.get_diff()
for line in diff.split('\n'):
if is_addition(line) and line.find('unsafe ') > -1:
self.warn(unsafe_warning_msg)
return
def on_pr_opened(self, api, payload):
diff = api.get_diff()
for line in diff.split('\n'):
if is_addition(line) and line.find('unsafe ') > -1:
self.warn(unsafe_warning_msg)
return