You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Oct 23, 2021. It is now read-only.
Currently everything lives in one big file with just a little abstraction. This makes maintenance a bit of a pain and contributions also less likely.
It would probably be beneficial to split out the code in several components. Something along the lines of:
start.js would be responsible for
discord.js would be responsible for the discord connection side of things.
irc-server.js would contain the connection functionality for the server.
utils.js would be the home of all the functions responsible for parsing data. Like these lines here and a lot of functionality that now happens directly in the event handlers themselves.
It would be no small task, but it has been on my mind for a while so I might as well make an issue for it.
The text was updated successfully, but these errors were encountered:
Currently everything lives in one big file with just a little abstraction. This makes maintenance a bit of a pain and contributions also less likely.
It would probably be beneficial to split out the code in several components. Something along the lines of:
start.js
would be responsible fordiscord.js
would be responsible for the discord connection side of things.irc-server.js
would contain the connection functionality for the server.utils.js
would be the home of all the functions responsible for parsing data. Like these lines here and a lot of functionality that now happens directly in the event handlers themselves.It would be no small task, but it has been on my mind for a while so I might as well make an issue for it.
The text was updated successfully, but these errors were encountered: