angular-docs-cn/dev-infra/caretaker/check
Paul Gschwendtner 67f65a9d25 refactor(dev-infra): improve type-safety of git client utility (#42468)
Currently the `GitClient` accepts a generic parameter for determining
whether the `githubToken` should be set or not. This worked fine so far
in terms of distinguishing between an authenticated and
non-authenticated git client instance, but if we intend to conditionally
show methods only for authenticated instances, the generic parameter
is not suitable.

This commit splits up the `GitClient` into two classes. One for
the base logic without any authorization, and a second class that
extends the base logic with authentication logic. i.e. the
`AuthenticatedGitClient`. This allows us to have specific methods only
for the authenticated instance. e.g.

  * `hasOauthScopes` has been moved to only exist for authenticated
    instances.
  * the GraphQL functionality within `gitClient.github` is not
    accessible for non-authenticated instances. GraphQL API requires
    authentication as per Github.

The initial motiviation for this was that we want to throw if
`hasOAuthScopes` is called without the Octokit instance having
a token configured. This should help avoiding issues as within
3b434ed94d
that prevented the caretaker process momentarily.

Additionally, the Git client has moved from `index.ts` to
`git-client.ts` for better discoverability in the codebase.

PR Close #42468
2021-06-03 14:34:33 -07:00
..
base.spec.ts refactor(dev-infra): use a singleton for GitClient (#41515) 2021-04-12 16:43:54 -07:00
base.ts refactor(dev-infra): improve type-safety of git client utility (#42468) 2021-06-03 14:34:33 -07:00
check.ts fix(dev-infra): set the default LogLevel of GitClient logging to DEBUG (#41899) 2021-05-07 10:15:20 -04:00
ci.spec.ts refactor(dev-infra): use a singleton for GitClient (#41515) 2021-04-12 16:43:54 -07:00
ci.ts fix(dev-infra): run caretaker checks asyncronously (#39086) 2020-11-03 16:32:59 -08:00
cli.ts refactor(dev-infra): use a singleton for GitClient (#41515) 2021-04-12 16:43:54 -07:00
g3.spec.ts refactor(dev-infra): improve type-safety of git client utility (#42468) 2021-06-03 14:34:33 -07:00
g3.ts refactor(dev-infra): improve type-safety of git client utility (#42468) 2021-06-03 14:34:33 -07:00
github.spec.ts refactor(dev-infra): improve type-safety of git client utility (#42468) 2021-06-03 14:34:33 -07:00
github.ts refactor(dev-infra): use a singleton for GitClient (#41515) 2021-04-12 16:43:54 -07:00
services.spec.ts refactor(dev-infra): use a singleton for GitClient (#41515) 2021-04-12 16:43:54 -07:00
services.ts fix(dev-infra): run caretaker checks asyncronously (#39086) 2020-11-03 16:32:59 -08:00