Skip to content

Commit e6dd83a

Browse files
authored
Support napi v2 (#111)
* Export a sensible user agent * Completely rewrite rust portions to be napi v2 compatible * Format rust files * Only lint TS files when running lint-node * Don't include autogenerated typings in the tree * changelog * Autofix generated rust dependencies due to a bug in napi@2 * Fix for multiple runs * Fix format_util file name * Make script generic * lint
1 parent 694d508 commit e6dd83a

21 files changed

+392
-491
lines changed

.github/workflows/main.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ jobs:
1818
with:
1919
node-version: 16
2020
- run: yarn --ignore-scripts
21-
- run: yarn lint
21+
- run: yarn lint:js
2222
lint-rust:
2323
runs-on: ubuntu-latest
2424
steps:

.gitignore

+4-1
Original file line numberDiff line numberDiff line change
@@ -11,4 +11,7 @@ public/
1111
# Added by cargo
1212
/target
1313

14-
book
14+
# Generated during build
15+
/src/libRs.d.ts
16+
17+
book

Cargo.lock

+142-43
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
[package]
22
name = "matrix-hookshot"
3-
version = "0.1.0"
3+
version = "1.0.0"
44
edition = "2021"
55

66
[lib]
77
crate-type = ["cdylib"]
88

99
[dependencies]
10-
napi = {version="1", features=["serde-json"]}
11-
napi-derive = "1"
10+
napi = {version="2", features=["serde-json"]}
11+
napi-derive = "2"
1212
url = "2"
1313
serde_json = "1"
1414
serde = "1"

build.rs

+2-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
1-
extern crate napi_build;
2-
31
fn main() {
4-
napi_build::setup();
2+
use napi_build::setup;
3+
setup();
54
}

changelog.d/111.misc

+1
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
Update to npai-rs@2

0 commit comments

Comments
 (0)