This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

Hm... PR builder is dead

@oleg-nenashev AFAIK, the test suite require local configuration. Was there a PR Builder configured in this repo?

Yep. It has been configured on Kohsuke's beescloud account. I'm looking for it

I have a mixed feeling about adding more connectXyz methods, but it is indeed convenient to have some of them. Given that @recena feels strongly about having this particular convenience method, I think it's OK to have it.

I've massaged this PR somewhat by the following commit.

@kohsuke Thanks for your time. I left GHApiInfo in a public class because we can need adding more fields in the future. I used isApiUrlValid() because we had isCredentialValid(), I tried to follow the rules.