Skip to content

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

Merged
daffl merged 3 commits intofeathersjs:dovefrom
FossPrime:patch-14
Apr 4, 2022
Merged

chore: Fix all package types fields and unify test command#2587
daffl merged 3 commits intofeathersjs:dovefrom
FossPrime:patch-14

Conversation

@FossPrime
Copy link
Copy Markdown
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
Copy Markdown
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.

2 participants