I'm in a process of setting a different default shipment method for gift cards. I've changed the return for

getGiftCardOnlyShipmentMethods

and so far it doesn't seem to be any issues anywhere, but i'm looking over the code for 'NoShipment' mentions, and there seems to be a bunch of sanitization and processing based on config value of NoShipment, some places even directly refer to string 'NoShipment', that said they're using GiftCardConfig shipment method instead of ShipmentConfig, the former being deprecated. Are there any places i should pay attention to or should the return function being changed handle it alone ?