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

feat(cli): allow bare specifier for builtin node module #20728

Merged
merged 25 commits into from
Oct 20, 2023
Merged

feat(cli): allow bare specifier for builtin node module #20728

Changes from 1 commit
Commits
Show all changes
25 commits
Select commit Hold shift + click to select a range
db03ccf
feat(cli): allow bare specifier for builtin node module
kt3k Sep 28, 2023
fccf6ae
refactor: use log::warn!
kt3k Sep 29, 2023
04961d8
show line and column numbers
kt3k Sep 30, 2023
6d37f6a
test: update lsp diagnostics expectation
kt3k Sep 30, 2023
100953b
show warning on bare node specifier via lsp
kt3k Sep 30, 2023
78063b9
remove dead code
kt3k Sep 30, 2023
a4308bb
Merge branch 'main' into allow-bare-specifier-for-builtin-node-modules
kt3k Sep 30, 2023
91aaf73
allow bare node specifier when import map is present
kt3k Sep 30, 2023
27de632
do not show diagnostics when bare node specifiers are explicitly mapp…
kt3k Sep 30, 2023
7c3a80a
test: update import map error expectations
kt3k Oct 1, 2023
f706dec
Merge branch 'main' into allow-bare-specifier-for-builtin-node-modules
kt3k Oct 2, 2023
ee7261e
use deno_graph 0.56.2, keep error from MappedSpecifierResolver
kt3k Oct 2, 2023
9380156
Merge branch 'main' into allow-bare-specifier-for-builtin-node-modules
kt3k Oct 4, 2023
91ff3ac
Merge branch 'main' into allow-bare-specifier-for-builtin-node-modules
kt3k Oct 5, 2023
2cbd685
add feature flag
kt3k Oct 5, 2023
517ec83
fix lsp test case
kt3k Oct 5, 2023
cab2bba
restore --quiet option in test
kt3k Oct 5, 2023
c8c5334
enable feature by config
kt3k Oct 5, 2023
bb26d6c
test lsp diagnostics with bare-node-builtins enabled by unstable fiel…
kt3k Oct 5, 2023
3b76e13
fix lint
kt3k Oct 5, 2023
8de32f5
Merge branch 'main' into allow-bare-specifier-for-builtin-node-modules
kt3k Oct 13, 2023
680b489
use feature_checker, implement CliOption#unstable_bare_node_builtins
kt3k Oct 13, 2023
accc12f
refactor: use map_or
kt3k Oct 13, 2023
54b6341
use self.options.unstable_bare_node_builtlins() directly in resolver …
kt3k Oct 13, 2023
9568be2
use map().unwrap_or() instead of map_or()
kt3k Oct 13, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix lint
  • Loading branch information
@kt3k
kt3k committed Oct 5, 2023
commit 3b76e13fc5c575e1eb7cbec12bd9e55c890ab23d
2 changes: 1 addition & 1 deletion cli/tests/integration/lsp_tests.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9994,7 +9994,7 @@ fn lsp_import_unstable_bare_node_builtins_auto_discovered() {
}));

let diagnostics = diagnostics
.messages_with_file_and_source(&main_script.to_string(), "deno")
.messages_with_file_and_source(main_script.as_ref(), "deno")
.diagnostics
.into_iter()
.filter(|d| {
Expand Down