party
main
Resolves #6. Updates #3.
@@ -16,0 +20,4 @@
private void OnEnable()
{
// NOTE(zeph): Party is mandatory. If none is set, we want an exception.
I think its clearer to make these types of notes into assertions. Assert.IsNotNull(party) or something like it.
Assert.IsNotNull(party)
@@ -16,0 +29,4 @@
party.Remove(this);
}
internal HeroPartySO getParty()
Should use a property instead of a getter
No dependencies set.
The note is not visible to the blocked user.
madxmike
Resolves #6.
Updates #3.
@@ -16,0 +20,4 @@private void OnEnable(){// NOTE(zeph): Party is mandatory. If none is set, we want an exception.I think its clearer to make these types of notes into assertions.
Assert.IsNotNull(party)or something like it.@@ -16,0 +29,4 @@party.Remove(this);}internal HeroPartySO getParty()Should use a property instead of a getter