Skip to content
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

Remove browserify dependency/bundler #7

Open
GrayedFox opened this issue Sep 12, 2019 · 0 comments
Open

Remove browserify dependency/bundler #7

GrayedFox opened this issue Sep 12, 2019 · 0 comments
Assignees
Labels
chore Refactoring, maintenance, fine tuning, all that jazz

Comments

@GrayedFox
Copy link
Owner

GrayedFox commented Sep 12, 2019

Project currently relies on browserify but it doesn't have to - the native extension way to "require" files is to break them up into content and background isolated logic and specify them in the manifest.json

  • would likely need to split the helpers file into smaller parts with dedicated content and background methods
  • would eliminate need to build/bundle files and make it easier to develop (can just watch and rebuild extension as normal)
  • good time to also restructure and refactor existing code base into distinct, modular parts
@GrayedFox GrayedFox added the chore Refactoring, maintenance, fine tuning, all that jazz label Sep 12, 2019
@GrayedFox GrayedFox self-assigned this Sep 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Refactoring, maintenance, fine tuning, all that jazz
Projects
None yet
Development

No branches or pull requests

1 participant