Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

address multiple issues found by pylint #206

Merged
merged 3 commits into from
Oct 26, 2020
Merged

address multiple issues found by pylint #206

merged 3 commits into from
Oct 26, 2020

Conversation

bmc-msft
Copy link
Contributor

  1. overwriting module imports (func and os)
  2. not using lazy-formatting in logging
  3. attempting to use lazy-formatting in exceptions
  4. invalid format string params (fixes Exception in resetting VMs #205 )
  5. returning values on funcs declared to return None

@bmc-msft bmc-msft mentioned this pull request Oct 26, 2020
Merged
@bmc-msft bmc-msft merged commit d4c5843 into microsoft:main Oct 26, 2020
@bmc-msft bmc-msft deleted the fix-pylint-errors branch October 26, 2020 16:24
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ranweiler ranweiler ranweiler approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Exception in resetting VMs
3 participants
@bmc-msft @ranweiler @demoray