Skip to content

Commit 88a79b9

Browse files
authored
Merge pull request #17 from bvanjoi/next
release: 0.0.12
2 parents 0d16fd1 + 458caaf commit 88a79b9

File tree

4 files changed

+14
-11
lines changed

4 files changed

+14
-11
lines changed

Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ crate-type = ["cdylib"]
1212
[dependencies]
1313
napi = "2"
1414
napi-derive = "2"
15-
nodejs-resolver = "0.0.19"
15+
nodejs-resolver = "0.0.21"
1616
serde = { version = "1.0.137", features = ["derive"] }
1717

1818
[build-dependencies]

index.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,4 +32,4 @@ export interface ResolverInternal {
3232

3333
}
3434
export function create(options: RawResolverOptions): ExternalObject<ResolverInternal>
35-
export function resolve(resolver: ExternalObject<ResolverInternal>, base_dir: string, id: string): string | false
35+
export function resolve(resolver: ExternalObject<ResolverInternal>, base_dir: string, id: string): string

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "nodejs-resolver",
3-
"version": "0.0.11",
3+
"version": "0.0.12",
44
"description": "node binding for nodejs-resolver",
55
"main": "index.js",
66
"license": "MIT",

src/lib.rs

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,7 @@ use napi::bindgen_prelude::External;
22
use napi_derive::napi;
33
use nodejs_resolver::{AliasMap, Resolver, ResolverOptions};
44
use serde::Deserialize;
5-
use std::{
6-
path::{Path, PathBuf},
7-
};
5+
use std::path::{Path, PathBuf};
86

97
#[derive(Debug, Clone, Deserialize)]
108
#[serde(rename_all = "camelCase")]
@@ -47,9 +45,9 @@ impl RawResolverOptions {
4745
.map_or(default.condition_names, |vec| vec.into_iter().collect()),
4846
symlinks: self.symlinks.unwrap_or(default.symlinks),
4947
description_file: self
50-
.description_file
51-
.to_owned()
52-
.unwrap_or(default.description_file),
48+
.description_file
49+
.to_owned()
50+
.unwrap_or(default.description_file),
5351
main_files: self.main_files.to_owned().unwrap_or(default.main_files),
5452
main_fields: self.main_fields.to_owned().unwrap_or(default.main_fields),
5553
prefer_relative: self.prefer_relative.unwrap_or(default.prefer_relative),
@@ -86,7 +84,7 @@ pub fn create(options: RawResolverOptions) -> Result<External<Resolver>, napi::E
8684

8785
#[napi(
8886
ts_args_type = "resolver: ExternalObject<ResolverInternal>, base_dir: string, id: string",
89-
ts_return_type = "string | false"
87+
ts_return_type = "string"
9088
)]
9189
pub fn resolve(
9290
resolver: External<Resolver>,
@@ -96,7 +94,12 @@ pub fn resolve(
9694
match (*resolver).resolve(Path::new(&base_dir), &id) {
9795
Ok(val) => {
9896
if let nodejs_resolver::ResolverResult::Info(info) = val {
99-
Ok(format!("{}{}{}", info.path.display(), &info.request.query, &info.request.fragment))
97+
Ok(format!(
98+
"{}{}{}",
99+
info.path.display(),
100+
&info.request.query,
101+
&info.request.fragment
102+
))
100103
} else {
101104
Ok(String::from("false"))
102105
}

0 commit comments

Comments
 (0)