I think it's a very laudable act (if action follows words).
I think in terms of the rest of Europe, and especially NATO countries in Europe, we should certainly be providing everything we can while still being careful to provide enough means for our own defence. This is especially true with the threat (however realistic) of another Trump presidency, which could leave us on our own in this regard.
Provided we have our own Europe-wide defence covered though, yes we should be giving all the aid we can to Ukraine.