AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

Remove some bogus noqa F841 comments

Open endolith opened this issue 2 years ago • 1 comments

Background

There is a noqa comment on e even though e is used and will not trigger QA. Other lines have noqa when it would be better to just not assign to a variable in the first place.

Changes

Removed the unused underscore variable assignments and unnecessary comments

Documentation

I removed erroneous comments

Test Plan

I tested it a bit, but I can't conceive of any negative effects. Removing assignment to an unused variable does nothing. Removing an erroneous comment does nothing.

PR Quality Checklist

  • [x] My pull request is atomic and focuses on a single change.
  • [x] I have thouroughly tested my changes with multiple different prompts.
  • [x] I have considered potential risks and mitigations for my changes.
  • [x] I have documented my changes clearly and comprehensively.
  • [x] I have not snuck in any "extra" small tweaks changes

endolith avatar Apr 12 '23 04:04 endolith

@endolith There are conflicts now

nponeccop avatar Apr 15 '23 14:04 nponeccop

Made irrelevant by 1073954f, but found another one and replaced this branch with that.

endolith avatar Apr 16 '23 12:04 endolith