do you think is best to get those constant strings into a const? in the interface?
we should also, in another PR, move the authentication methods to the interface and not the github adapter