search

ynput / ayon-core

Apache License 2.0
28 stars 34 forks source link

Chore: Validate Content plugin settings 2 #530

Closed iLLiCiTiT closed 4 months ago

iLLiCiTiT commented 4 months ago

Changelog Description

Rename ValidateContainers to ValidateOutdatedContainers.

dee-ynput commented 4 months ago

Do we have a reason to not use "Validate Outdated Containers" for the title @iLLiCiTiT ?

iLLiCiTiT commented 4 months ago

Do we have a reason to not use "Validate Outdated Containers" for the title @iLLiCiTiT ?

No, the suggestion was added after merge.

dee-ynput commented 4 months ago

Do we have a reason to not use "Validate Outdated Containers" for the title @iLLiCiTiT ?

No, the suggestion was added after merge.

Yep. So, if I'm following correctly, this second PR was to add the changes suggested after the first merge. And now is time for a third one to actually include comment about the title.

I guess my question is: "who's going to make that 3rd PR ?"

Sorry for passive agressive sarcasm... -_____-

BigRoy commented 4 months ago

That'll be me.

Confusing because I had to check both the Me box and the Roy box - hehe.