-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
main.js is a mess #2
Comments
this is about the |
Partially, but overall that file is rather badly structured and only really does the bare minimum. |
Alright.
each of this rule is a single js file. Each of the rule of course do a single functionality. Now for this case, if you want, main.js can be split into rules like Or we can just follow Slack Notification Simple workflow, which contains only a single file |
The entire file needs a refactor at some point. It definitely works as intended but it's slightly embarrassing.
The text was updated successfully, but these errors were encountered: