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

refactor: load module content an owned value #154

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Commits on May 20, 2022

  1. fix: error on conflicting assertions

    Previously, if a specifier was imported more than once, with conflicting
    assertions, we'd only really consider the final import. This is wrong:
    conflicting assertions are an error in-themselves already. I think this
    behaviour is still wrong, because if a module is imported with incorrect
    assertions later in the graph after the module has already been loaded by
    an import with a correct assertion, no error will be raised.
    lucacasonato committed May 20, 2022
    Configuration menu
    Copy the full SHA
    cbf7134 View commit details
    Browse the repository at this point in the history
  2. refactor: load module content an owned value

    This is a pre-requisite for WASM imports that additionally allows us to
    minimize clones when doing things like BOM stripping.
    lucacasonato committed May 20, 2022
    Configuration menu
    Copy the full SHA
    024c08e View commit details
    Browse the repository at this point in the history

Commits on May 30, 2022

  1. Configuration menu
    Copy the full SHA
    69643d5 View commit details
    Browse the repository at this point in the history
  2. lint

    bartlomieju committed May 30, 2022
    Configuration menu
    Copy the full SHA
    6b6c5d7 View commit details
    Browse the repository at this point in the history