FIX: avoid index error in vlan accumulation

[ZLLentz]

Description

Fix an issue with parsing vlan names without "-" in them- for now, ignore these
Try to keep the overall behavior the same, including order added to the list

Also run pre-commit to solve one of the CI build errors

Motivation and Context

This was crashing the gui for xpp

$ switchtool -s swh-xpp-h3
Traceback (most recent call last):
  File "/cds/group/pcds/pyps/apps/switchtool/R2.0.0/switchtool/ui/widgets/switch.py", line 330, in refresh
    if v not in vl and allvlan[v].split("-")[1] == "CDS":
                       ~~~~~~~~~~~~~~~~~~~~~^^^
IndexError: list index out of range
/cds/group/pcds/pyps/apps/switchtool/latest/scripts/switchtool: line 17: 1058446 Aborted                 (core dumped) python "$RELDIR"/scripts/switch_gui.py "$@"

How Has This Been Tested?

Interactively only

Where Has This Been Documented?

Here only

Pre-merge checklist

• Code works interactively
• Code follows the style guide
• Code contains descriptive docstrings, including context and API
• New/changed functions and methods are covered in the test suite where possible
• Test suite passes locally
• Test suite passes on GitHub Actions
• Ran docs/pre-release-notes.sh and created a pre-release documentation page
• Pre-release docs include context, functional descriptions, and contributors as appropriate

[tangkong]

LGTM