Minor changes

[acenolaza]

Nothing major since it is just a trimmed down version of StakeableVesting. Just a few minor things that can be ignored.

@bbenligiray one concern that I have is if anyone ever sends more tokens than amount from initialize() to this contract then those tokens might get locked forever, right?

[bbenligiray]

Yeah but the user is explicitly told to use IrrevocableVestingFactory to deploy IrrevocableVesting and it's to be expected for the user to rek themselves if they don't follow the instructions

[bbenligiray]

I pushed some of these to main and closing the PR but we can continue the discussions above

[acenolaza]

Yeah but the user is explicitly told to use IrrevocableVestingFactory to deploy IrrevocableVesting and it's to be expected for the user to rek themselves if they don't follow the instructions

Yes that's true but I was thinking more of the case where the use the factory but then someone mistakenly send more funds to the contract directly in order to increase the vested amount. I know this is unlikely to happen for our use case so we can end the discussion here if you think so

[bbenligiray]

Ah got it. It feels too random of a user error to handle specifically though.