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.

Sign up for GitHub

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

Jump to bottom

fix(transforms): var conflicts in cjs-to-esm require-to-import transform #52

Merged
merged 2 commits into from
May 2, 2023
Merged

fix(transforms): var conflicts in cjs-to-esm require-to-import transform #52

merged 2 commits into from
May 2, 2023

Commits on Apr 28, 2023

  1. fix(transforms): var conflicts in cjs-to-esm require-to-import transform 

    When name of imported module is same as name of variable in scope, it
causes a conflict. This commit fixes that by renaming the imported
module to `${pckgName}Module`, if it's a default import.

When the imported module is a named import, it's renamed to
`${pckgName}${importedName}`.
    @js07
    js07 committed Apr 28, 2023
    Configuration menu
    Copy the full SHA
    4015cf8 View commit details
    Browse the repository at this point in the history

  2. refactor: style: rename vars in cjs-to-esm.js for readability

    @js07
    js07 committed Apr 28, 2023
    Configuration menu
    Copy the full SHA
    994c944 View commit details
    Browse the repository at this point in the history