search

thuiop / magium-dev

MIT License
13 stars 4 forks source link

Code cleanup #50

Closed ZiClaud closed 2 months ago

ZiClaud commented 2 months ago

Issue #45

himanshunaidu commented 2 months ago

There are far too many different kinds of changes being put together in one PR here. We are all already working on separate PRs for and this is going to conflict a lot with every one of those. I get the idea behind the issue you raised, but we need to be very careful while trying to push these changes. It'd have been better if we raised separate PRs for linting issues, splitting the css files, removing main_node and other files (which I am not sure are redundant btw, please check with @thuiop on this), etc.

ZiClaud commented 2 months ago

Alright, will work on fixing it, also I'll add some documentation on main_node.js

ZiClaud commented 2 months ago

Will create a new Pull Request