Age | Commit message (Collapse) | Author |
|
|
|
Fixes #13437
|
|
|
|
Fixes: #13383
Fixes: denoland/vscode_deno#609
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
Co-authored-by: Erfan Safari <erfanshield@outlook.com>
|
|
Closes: #13272
|
|
|
|
|
|
|
|
|
|
|
|
Fixes: #13216
|
|
|
|
Closes #13153
|
|
Closes: #13130
|
|
|
|
|
|
Fixes: #13099
|
|
|
|
This commit adds proper support for import assertions and JSON modules.
Implementation of "core/modules.rs" was changed to account for multiple possible
module types, instead of always assuming that the code is an "ES module". In
effect "ModuleMap" now has knowledge about each modules' type (stored via
"ModuleType" enum). Module loading pipeline now stores information about
expected module type for each request and validates that expected type matches
discovered module type based on file's "MediaType".
Relevant tests were added to "core/modules.rs" and integration tests,
additionally multiple WPT tests were enabled.
There are still some rough edges in the implementation and not all WPT were
enabled, due to:
a) unclear BOM handling in source code by "FileFetcher"
b) design limitation of Deno's "FileFetcher" that doesn't download the same
module multiple times in a single run
Co-authored-by: Kitson Kelly <me@kitsonkelly.com>
|
|
|
|
* fix(lsp): handle import specifier not having a trailing quote
* clean up
* Add test.
|
|
Resolves #10051
|
|
Co-authored-by: Bartek Iwańczuk <biwanczuk@gmail.com>
|
|
Fixes #9931
|
|
|
|
|
|
|
|
|
|
publishDiagnostics messages (#12868)
The tests expect 3 publish notifications. It was possible for less than 3 to occur if two or more tasks set the diagnostics in the collection, exited the lock at the same time, then called `publish_diagnostics`
|
|
|
|
* Add `specifier_to_file_path` to support converting a ModuleSpecifier with a unix-style path to a PathBuf on Windows.
|
|
|
|
|
|
|
|
mutated to change it (#12747)
|
|
|
|
Fixes #12753
|
|
|
|
|
|
(#12663)
|
|
Closes #8440
|