Simplify type checking in sort_collections() Closes https://github.com/StackGuardian/tirith/issues/185

[gabrilito1]

Update the sort_collections function in the utils.py to use a single isinstance check for primitive types.

[refeed]

Hi @gabrilito1 , thanks this looks good.

Only a few minor things though:

1. Could you fix the linting warnings by running black in your system?
2. Could you make your commit message into:

Simplify type checking in sort_collections()

Closes https://github.com/StackGuardian/tirith/issues/185

[gabrilito1]

Done, give me a feedback asap

[refeed]

Looks good, I'll just do squash and rebase on this commit. Thanks!

[refeed]

BTW @gabrilito1 , I forgot to mention that what I meant by the commit message is the git commit message. Ideally, we could squash the commits into one and then use git commit --amend to change the commit message and then do force push again. But for now, it's fine, we'll do it on my side, but if you want to change the commit message, feel free to do so

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[samriddhi99]

LGTM