Comments (2)
I guess you could just make a pr if you really care about it. The TypeScript documentation is pretty outdated and the reasons they have listed there are not really true anymore.
Edit: Just to elaborate, type aliases do show up in errors, they can be implemented just like interfaces and they can be "extended" with intersections.
from discord.js-typings.
That's not entirely true in this case. Having a name that can be referred to rather than an object literal in cases of a passed value not meeting the shape of the required type can mean a huge difference in compiler error legibility.
Additionally, I personally prefer to strictly use interfaces for things that will actually be implemented, but I'm becoming more open to using interfaces for other purposes in my own code lately.
That said, this is a case for a PR rather than an issue. I can tell you now I don't want to have to go back over all the work I've put into this and convert types to interfaces.
If you want to prepare your own PR for this you're more than welcome to. I'm not going to leave this issue open in the meantime because it's far from pertinent and I don't foresee anyone else on the team wanting to take the time to tackle this either.
from discord.js-typings.
Related Issues (18)
- Make node types optional HOT 3
- DMChannel.send() return type wrong? HOT 1
- Guild.member may return undefined HOT 4
- MessageOptions property 'code' should be type 'string' | 'boolean' HOT 1
- Add 'How to use' section in readme HOT 2
- Missing "Constants" export
- package.json missing name HOT 2
- Contribute to Definetely Typed HOT 3
- Namespace naming conflicts and inconsistency HOT 5
- discord.js 11.3.0 errors HOT 3
- node_modules/discord.js/typings/index.d.ts(267,40): error TS2689: Cannot extend an interface 'Map'. Did you mean 'implements'? HOT 11
- Cannot get typings to load HOT 2
- Turn this repostory into a npm @types package. HOT 1
- MessageOptions.files should be updated HOT 1
- Missing definition for GuildMemberRoleStore#sweep HOT 1
- createMessageCollector() options should be MessageCollectorOptions instead of CollectorOptions HOT 1
- Make this a separate git repo HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from discord.js-typings.