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

chore: add support for commonJS to esm #15

Merged
merged 6 commits into from
Nov 27, 2023

Conversation

ninamarina
Copy link
Contributor

@ninamarina ninamarina commented Nov 17, 2023

Jira ticket: https://dialpad.atlassian.net/browse/DLT-1288

Description

Adds support for require commonJS modules to esm to avoid code duplication.
Plugin docs: https://github.com/originjs/vite-plugins/tree/main/packages/vite-plugin-commonjs

Pull Request Checklist

  • Ask the contributors if a similar effort is already in process or has been solved.
  • Review the contribution guidelines.
  • Use staging as your pull request's base branch. (All PRs using production as its base branch will be declined).
  • Ensure all gulp scripts successfully compile.
  • Update, remove, or extend all affected documentation.
  • Ensure no private Dialpad links or info are in the code or pull request description (Dialtone is a public repo!).

Obligatory GIF (super important!)

200w (3)

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-15
🔨 If you experience an SSL issue then wait 2 minutes and try again.

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-15
🔨 If you experience an SSL issue then wait 2 minutes and try again.

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-15
🔨 If you experience an SSL issue then wait 2 minutes and try again.

@iropolo
Copy link
Contributor

iropolo commented Nov 21, 2023

Copy link

✔️ Deploy Preview ready!
😎 Browse the preview: https://dialpad.design/deploy-previews/pr-15
🔨 If you experience an SSL issue then wait 2 minutes and try again.

@ninamarina ninamarina marked this pull request as ready for review November 24, 2023 13:16
@ninamarina ninamarina merged commit 6fa1401 into staging Nov 27, 2023
8 checks passed
@ninamarina ninamarina deleted the add-commonjs-transform-plugin branch November 27, 2023 13:51
@ninamarina ninamarina mentioned this pull request Nov 27, 2023
6 tasks
ninamarina added a commit that referenced this pull request Nov 29, 2023
Updates the linting links in the CONTRIBUTING.md to be correct after #15
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.

3 participants