This PR makes a few changes to FlagValueSource and NonSendableFlagValueSource:
Adds an explicit flagValueSourceID property instead of trying to piggy back off Identifiable.
Renames name to flagValueSourceName to better disambiguate with other existing properties called name that a type you are trying to conform to FlagValueSource might have.
Renames changes to flagValueChanges to better disambiguate with other existing properties called changes that a type you are trying to conform to FlagValueSource might have.
These changes resolve some errors building with Xcode 16 but are also just being a good citizen. We shouldn't be trying to force Identifiable on a type that just wants to be a FlagValueSource when all we need is an ID we can use for managing subscriptions.
📒 Description
This PR makes a few changes to
FlagValueSource
andNonSendableFlagValueSource
:flagValueSourceID
property instead of trying to piggy back offIdentifiable
.name
toflagValueSourceName
to better disambiguate with other existing properties calledname
that a type you are trying to conform toFlagValueSource
might have.changes
toflagValueChanges
to better disambiguate with other existing properties calledchanges
that a type you are trying to conform toFlagValueSource
might have.These changes resolve some errors building with Xcode 16 but are also just being a good citizen. We shouldn't be trying to force
Identifiable
on a type that just wants to be aFlagValueSource
when all we need is an ID we can use for managing subscriptions.This is part of the Vexil 3 alpha series.