STIMPACK both in buff_id and upgrade_id

[tweakimp]

The variable STIMPACK is both in buff_id and upgrade_id. That means in sc2.constants one of them will be overwritten.

[BurnySc2]

This is only true if you do

from sc2.constants import *

You can avoid it by doing:

from sc2.ids.unit_typeid import UnitTypeId
from sc2.ids.ability_id import AbilityId
from sc2.ids.buff_id import BuffId
from sc2.ids.upgrade_id import UpgradeId
from sc2.ids.effect_id import EffectId

This is probably the better choice because VScode and pycharm both mark the enums red if you dont have a UnitTypeId. in front of the name, unless you disabled that warning.

Also, see #58 and #55

Sidenote:
There isn't anything we can do since these are the names directly from the SC2 API.

[tweakimp]

Ah yeah you are right. Is is done in many examples like the mass reaper one, i thought it was supposed to be that way.