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

ES Modules #71

Merged
merged 12 commits into from
Oct 13, 2019
Merged

ES Modules #71

merged 12 commits into from
Oct 13, 2019

Conversation

yorkxin
Copy link
Owner

@yorkxin yorkxin commented Oct 13, 2019

Summary

Converted to ES Modules for the following reasons:

  1. Both Chrome and Firefox added support to ESM
  2. Firefox disapproved my add-on because I am "delivering obscuring code" which was generated by webpack

Webpack has been removed from the build pipeline, which was used to inject build environment flags. Because those environment flags are not available anymore, the program has to detect / sniff the current supported browser features, especially to copy the text.

Also refactored many code to ensure that it works on both browsers.

Also added ESLint.

Tests

  • Chrome stable (macOS)
  • Firefox stable (macOS)
  • Chrome stable (Windows)
  • Firefox stable (Windows)

Optional:

  • Chrome beta (macOS)
  • Firefox beta (macOS)
  • Chrome beta (Windows)
  • Firefox beta (Windows)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant