addressing review comments

Signed-off-by: Brian Fletcher <brian@roadie.io>

packages/integration/api-report.md

@@ -203,11 +203,6 @@ export interface GithubCredentialsProvider {
   getCredentials(opts: { url: string }): Promise<GithubCredentials>;
 }
 
-// @public
-export type GithubCredentialsProviderFactory = (
-  config: GitHubIntegrationConfig,
-) => GithubCredentialsProvider;
-
 // @public
 export type GithubCredentialType = 'app' | 'token';
 
@@ -432,7 +427,7 @@ export class SingleInstanceGithubCredentialsProvider
   implements GithubCredentialsProvider
 {
   // (undocumented)
-  static create: GithubCredentialsProviderFactory;
+  static create: (config: GitHubIntegrationConfig) => GithubCredentialsProvider;
   getCredentials(opts: { url: string }): Promise<GithubCredentials>;
 }
 

packages/integration/src/github/SingleInstanceGithubCredentialsProvider.ts

@@ -22,7 +22,6 @@ import { DateTime } from 'luxon';
 import {
   GithubCredentials,
   GithubCredentialsProvider,
-  GithubCredentialsProviderFactory,
   GithubCredentialType,
 } from './types';
 
@@ -232,7 +231,9 @@ export class GithubAppCredentialsMux {
 export class SingleInstanceGithubCredentialsProvider
   implements GithubCredentialsProvider
 {
-  static create: GithubCredentialsProviderFactory = config => {
+  static create: (
+    config: GitHubIntegrationConfig,
+  ) => GithubCredentialsProvider = config => {
     return new SingleInstanceGithubCredentialsProvider(
       new GithubAppCredentialsMux(config),
       config.token,

packages/integration/src/github/index.ts

@@ -27,7 +27,6 @@ export {
 export type {
   GithubCredentials,
   GithubCredentialsProvider,
-  GithubCredentialsProviderFactory,
   GithubCredentialType,
 } from './types';
 export { GitHubIntegration, replaceGitHubUrlType } from './GitHubIntegration';

packages/integration/src/github/types.ts

@@ -14,8 +14,6 @@
  * limitations under the License.
  */
 
-import { GitHubIntegrationConfig } from './config';
-
 /**
  * The type of credentials produced by the credential provider.
  *
@@ -43,13 +41,3 @@ export type GithubCredentials = {
 export interface GithubCredentialsProvider {
   getCredentials(opts: { url: string }): Promise<GithubCredentials>;
 }
-
-/**
- * This allows implementations to provide factories to create credential providers
- *
- * @public
- *
- */
-export type GithubCredentialsProviderFactory = (
-  config: GitHubIntegrationConfig,
-) => GithubCredentialsProvider;