Skip to content
This repository was archived by the owner on Aug 22, 2023. It is now read-only.

fix parse F constraint - #350

Open
marcj wants to merge 1 commit into
oclif:mainfrom
marcj:patch-1
Open

marcj wants to merge 1 commit into
oclif:mainfrom
marcj:patch-1

Conversation

@marcj

@marcj marcj commented Nov 25, 2022

Copy link
Copy Markdown

This makes it compatible with TypeScript 4.8

before this change you get an error:

packages/app/node_modules/@oclif/command/lib/command.d.ts:70:31 - error TS2344: Type 'F' does not satisfy the constraint 'Output'.

70     }>(options?: Parser.Input<F>, argv?: string[]): Parser.Output<F, A>;
                                 ~

This makes it compatible with TypeScript 4.8

before this change you get an error:

```
packages/app/node_modules/@oclif/command/lib/command.d.ts:70:31 - error TS2344: Type 'F' does not satisfy the constraint 'Output'.

70     }>(options?: Parser.Input<F>, argv?: string[]): Parser.Output<F, A>;
                                 ~
```
@git2gus

git2gus Bot commented Nov 25, 2022

Copy link
Copy Markdown

Git2Gus App is installed but the .git2gus/config.json doesn't exist.

@salesforce-cla

Copy link
Copy Markdown

Thanks for the contribution! Before we can merge this, we need @marcj to sign the Salesforce.com Contributor License Agreement.

@marcj

marcj commented Nov 28, 2022

Copy link
Copy Markdown
Author

Signing the agreement gave me "Could not sign the CLA, please contactoss-cla@salesforce.com", when I tried again it says "You already signed the CLA on 2022-11-28"🤷‍♂️

This branch has not been deployed

No deployments
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant