Skip to content

Commit

Permalink
[FABCN-430] Fix type for timestamp.second (#194) (#209)
Browse files Browse the repository at this point in the history
This patch fixes the type definition for the second field in the
Timestamp, which is actually Long (int64).

Signed-off-by: Taku Shimosawa <taku.shimosawa@hal.hitachi.com>

Co-authored-by: Matthew B White <mbwhite@users.noreply.github.com>

Co-authored-by: Taku Shimosawa <taku.shimosawa@hal.hitachi.com>
  • Loading branch information
mbwhite and shimos authored Sep 17, 2020
1 parent 85188f5 commit fc32346
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 7 deletions.
7 changes: 5 additions & 2 deletions apis/fabric-shim-api/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -22,5 +22,8 @@
"eslint": "6.6.0"
},
"types": "./types/index.d.ts",
"license": "Apache-2.0"
}
"license": "Apache-2.0",
"devDependencies": {
"@types/long": "^4.0.1"
}
}
3 changes: 2 additions & 1 deletion apis/fabric-shim-api/types/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,9 +5,10 @@
*/
declare module 'fabric-shim-api' {
import Long = require("long");

interface Timestamp {
seconds: number;
seconds: Long;
nanos: number;
}

Expand Down
17 changes: 13 additions & 4 deletions common/config/rush/pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit fc32346

Please sign in to comment.