search

misode / brigadier-ts

TypeScript port of Mojang/brigadier
MIT License
18 stars 5 forks source link

Big integer argument types should be available #7

Closed haykam821 closed 2 years ago

haykam821 commented 2 years ago

Right now, BigInt cannot be expressed through one of the existing argument type short of manually parsing it from a string.

misode commented 2 years ago

I'm not sure if I want to include this in the package, since support for BigInt is not the best yet. Would it be acceptable to create this ArgumentType yourself?

haykam821 commented 2 years ago

Would it make sense to have a separate package dedicated to bleeding-edge or less common types? I can easily create an argument type myself but I would rather not have to maintain a package to share the argument type across projects.

misode commented 2 years ago

I presume there probably isn't a project that uses this with old browsers as a requirement, so it should be fine to add this.