Skip to content
This repository has been archived by the owner on Sep 23, 2021. It is now read-only.

Add TypeScript typings file #115

Merged
merged 1 commit into from Nov 13, 2019
Merged

Add TypeScript typings file #115

merged 1 commit into from Nov 13, 2019

Conversation

bpasero
Copy link
Contributor

@bpasero bpasero commented Nov 7, 2019

This allows VSCode to remove our sudo-prompt.d.ts in our repo.

Refs: microsoft/vscode#83421

@bpasero
Copy link
Contributor Author

bpasero commented Nov 12, 2019

@jorangreef ping

@jorangreef jorangreef merged commit 3d6f25b into jorangreef:master Nov 13, 2019
@jorangreef
Copy link
Owner

Thanks @bpasero!

@bpasero bpasero deleted the types branch November 13, 2019 06:57
@bpasero
Copy link
Contributor Author

bpasero commented Nov 13, 2019

@jorangreef thanks, can you publish a minor new release so that VSCode can consume?

@jorangreef
Copy link
Owner

Done!

@bpasero
Copy link
Contributor Author

bpasero commented Nov 14, 2019

Thanks, can you publish to NPM too?

image

@jorangreef
Copy link
Owner

Sorry about that @bpasero, I did but didn't see that the publish failed. I have redone it now.

@bpasero
Copy link
Contributor Author

bpasero commented Nov 14, 2019

@jorangreef is it possible that some npmignore excludes it? I do not see the file:

image

@jorangreef
Copy link
Owner

Good catch. The package files whitelist also needed to be updated.

A patch version to fix this has just been published to NPM.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants