isort

[Itay4]

Status

• Ready

Description

• Added isort to pre-commit hooks and extractor
• Fixed a regression from Split yml powershell support #316 in which the the suffix .py was not added to the file name and autopep8 got wrong file name

Must have

• Tests

[glicht]

Good catch

[Itay4]

@glicht ive sorted the imports in all the repo, if you would like to review again

[coveralls]

Pull Request Test Coverage Report for Build 2290

• 182 of 224 (81.25%) changed or added relevant lines in 57 files are covered.
• 1 unchanged line in 1 file lost coverage.
• Overall coverage increased (+0.2%) to 56.338%

---

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
demisto_sdk/commands/common/hook_validations/classifier.py	0	1	0.0%
demisto_sdk/commands/run_playbook/playbook_runner.py	0	1	0.0%
demisto_sdk/commands/common/hook_validations/description.py	0	2	0.0%
demisto_sdk/commands/split_yml/extractor.py	7	9	77.78%
demisto_sdk/commands/init/initiator.py	0	6	0.0%
demisto_sdk/commands/validate/file_validator.py	0	14	0.0%
demisto_sdk/main.py	0	16	0.0%

Files with Coverage Reduction	New Missed Lines	%
demisto_sdk/commands/validate/file_validator.py	1	0%

Totals
Change from base Build 2288:	0.2%
Covered Lines:	4262
Relevant Lines:	7565

---

💛 - Coveralls