Is it a good practice to have logger as a singleton?

I put a logger instance in my dependency injection container, which then injects the logger into the classes which need one.