No, it's not. Because it doesn't add anything. Just document if something is mutating and what it mutates. Or simply name your functions accordingly. Like of

No, it's not. Because it doesn't add anything. Just document if something is mutating and what it mutates. Or simply name your functions accordingly. Like of your setter is doing a HTTP call, maybe name it so that it's obvious.