Closed ChrisChinchilla closed 1 year ago
This pull request fixes 1 alert when merging 34b98be0dce0ab9b5a7cebb902081b7a525738bb into 5754e3d9d5fe7e0714b1ce72d2c42d99da767d94 - view on LGTM.com
fixed alerts:
OK @jdkato Try the latest changes. it also made me realise that a seperate change is to go through a lot of the older rules and add actions to them. Once this is merged I'll also look at spelling.
This pull request fixes 1 alert when merging 2d08a6649fdab2bd56e56a54482570f096b9aa05 into 677fb65ca3d54f3fdbaabe98a4f2b6710a1ba794 - view on LGTM.com
fixed alerts:
Looks good. I'm going to test it a few days before I make a release, just to be sure.
@jdkato Sure, we can even squeeze some other features in to a release too. I'l' also slowly start updating rules with actions.
This pull request fixes 1 alert when merging b6b8b375f8bc661dba37bbf27b68d27424c01748 into 5754e3d9d5fe7e0714b1ce72d2c42d99da767d94 - view on LGTM.com
fixed alerts: