Looks fine. Mayyyyyyybe the 1k resistor is a little high, depends on the gain of the transistor. 5 volts would feed it with 5mA on its base, which might not be enough to fully switch it on. However, as it's only driving the relatively high resistance coil in the relay it's nothing to worry about if it works.
If you were driving it with 3.3V, I'd probably swap that transistor out for a Darlington pair to make sure the gain is high enough to fully switch on at that kind of drive voltage. You can get Darlington pairs in the same pinout as normal transistors so it would be a straight swap on the board.
Having said all that, if it's working in your application, leave it. 🤷