feat: type documented events

[Quantumlyy]

No description provided.

[github-actions]

Coverage increased (+1.4%) to 29.032% when pulling 88c6056 on feat/types/events into 7f7be02 on main.

[kyranet]

Just one question, can people still do client.on('unload', cb) and still get cb typed?

[Quantumlyy]

Just one question, can people still do client.on('unload', cb) and still get cb typed?

1. It will be typed but with one downside, store will be unknown as the generic in Store does not default to anything. so it had to be unknown.
2. The Events enum will have to be used, but it should be fine for JS users.

[kyranet]

It will be typed but with one downside, store will be unknown as the generic in Store does not default to anything. so it had to be unknown.

You can define store as Store<Piece>, it's not abstract, so the type is accepted.

[Quantumlyy]

It will be typed but with one downside, store will be unknown as the generic in Store does not default to anything. so it had to be unknown.

You can define store as Store<Piece>, it's not abstract, so the type is accepted.

Already tried it, but I will push it so you can see the error since it says Store does not fit Store.

[kyranet]

In that case, go to those lines:

https://github.com/sapphire-project/framework/blob/92525ad49947ca3af7c27c2cbf6f1744539e226b/src/lib/structures/base/BaseAliasStore.ts#L13-L14
https://github.com/sapphire-project/framework/blob/92525ad49947ca3af7c27c2cbf6f1744539e226b/src/lib/structures/base/BaseStore.ts#L13-L14

And make them store as unknown as Store<Piece>.