I ran ESLint v6.0.13.0 locally, and ran `eslint --fix` on the `ai/petra/` directory to automatically fix all violations of the ESLint rules in our eslintrc.json file that could be automatically fixed.
With D2070 and D2103 applied firstThere were also a number of warnings that could not be fixed automatically, this means thwhich were are now 0 violations of any kind in this sub directory with our current ESLint configurationfactored out in D2070 and D2103 and merged first.
In addition to the automatic fixes, this commit also two lines of commented out code that were oddly auto-formatted due to being commented out inline instead of as whole statements. Rather than fixing the indentation, this one case in `AttackPlan#checkTargetObstruction` I removed as it had been commented out from the first commit it was introduced (r18257), and didn't seem to have an obvious purposeThis means there should now be 0 ESLint violations in the ai/petra directory.
Depends on D2070In addition to the automatic fixes, this commit also two lines of commented out code that were oddly auto-formatted due to being commented out inline instead of as whole statements. Rather than fixing the indentation to adhere to ESLInt, I removed it in `AttackPlan#checkTargetObstruction`. It had been commented out from its first commit (r18257), and didn't seem to have an obvious purpose.
Depends on D2103.