Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Fix all package types fields and unify test command #2587

Merged
merged 3 commits into from
Apr 4, 2022
Merged

chore: Fix all package types fields and unify test command #2587

merged 3 commits into from
Apr 4, 2022

Conversation

FossPrime
Copy link
Member

The NPM package page thinks this package is missing types... it's not. This PR makes it show the TS symbol next to the name.

https://www.npmjs.com/package/@feathersjs/koa

Should have the same TS symbol that the main package does...
Screenshot 2022-03-30 11 17 14

The NPM package page thinks this package is missing types... it's not. This PR makes it show the TS symbol next to the name. 

https://www.npmjs.com/package/@feathersjs/koa

Should have the same TS symbol that the main package does...
@daffl
Copy link
Member

daffl commented Apr 1, 2022

Whoops, thanks! I think I at least forgot it in schema and memory as well. Would you mind checking all the package.json and add it anywhere else it may have been missed?

@daffl daffl changed the title Add TS support hint for NPM chore: Fix all package types fields and unify test command Apr 4, 2022
@daffl daffl merged commit c7612d1 into feathersjs:dove Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants