Skip to content

Commit

Permalink
Merge pull request #38 from TinkoffCreditSystems/fix/redis-set-allow-…
Browse files Browse the repository at this point in the history
…return-null

fix: redis: allow null return from set
  • Loading branch information
alexkvak authored Nov 17, 2020
2 parents 04f0b47 + 668908a commit 50fa3ca
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ jobs:
- uses: actions/checkout@v2

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/[email protected].2
uses: actions/[email protected].4
with:
node-version: ${{ matrix.node-version }}
# Optional registry to set up for auth. Will set the registry in a project level .npmrc and .yarnrc file, and set up auth to read in from env.NODE_AUTH_TOKEN
Expand Down Expand Up @@ -87,7 +87,7 @@ jobs:
persist-credentials: false

- name: Setup Node.js
uses: actions/[email protected].2
uses: actions/[email protected].4
with:
node-version: 10.x

Expand Down
2 changes: 1 addition & 1 deletion src/adapters/RedisStorageAdapter.ts
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ interface Redis extends EventEmitter {
expiryMode?: string,
time?: number | string,
setMode?: number | string
): Promise<string>;
): Promise<"OK" | null>;

mget(...keys: string[]): Promise<Array<string | null>>;

Expand Down

0 comments on commit 50fa3ca

Please sign in to comment.