search

CRBT-Team / Purplet

A framework for quickly building Discord bots + more packages relating to the Discord API, with a primary goal on a rich developer experience.
https://purplet.js.org
Apache License 2.0
64 stars 1 forks source link

Setup ESLint #18

Closed paperdave closed 2 years ago

paperdave commented 2 years ago

eslint is setup, however we do not use it due to https://github.com/davecaruso/various-web/issues/1, which is from my personal eslint config. i never set it up to use typescript properly so we get a lot of lint errors when it comes to it.

once that closes i'll setup eslint project wide

Hycord commented 2 years ago

could we use a popular eslint config like airbnb, or just create a new eslint config from scratch specifically for this project, and get it fully, 100% fleshed out to the point that literally any project using eslint could use it and it wouldn't cause a bunch of stupid errors?

paperdave commented 2 years ago

what you're suggesting is basically what i want to do except it's not in this repository. since other people are going to be using it now, I really really should go and get the config fully fleshed out. I'll put my priority there since a majority of my work to the rest client is done

paperdave commented 2 years ago

got my stuff updated, might be slightly buggy. i'll probably merge it in once we get rest done since it's a big pr

paperdave commented 2 years ago

BTW, im waiting for a few of my branches to merge before I fully enable this (and fix every lint issue without merge conflicts) but it's on my radar.