Skip to content

Commit a655df2

Browse files
authored
Merge pull request #1664 from tediousjs/dependabot/npm_and_yarn/multi-263f7b4ea4
chore(deps): bump @azure/identity and tedious
2 parents cae254f + c56a458 commit a655df2

File tree

4 files changed

+257
-193
lines changed

4 files changed

+257
-193
lines changed

.github/workflows/nodejs.yml

+6-4
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ jobs:
2222
- name: Setup Node.js
2323
uses: actions/setup-node@v4
2424
with:
25-
node-version: 16.x
25+
node-version: 18.x
2626
cache: 'npm'
2727
- name: Install dependencies
2828
run: npm clean-install
@@ -43,7 +43,7 @@ jobs:
4343
- name: Setup Node.js
4444
uses: actions/setup-node@v4
4545
with:
46-
node-version: 16.x
46+
node-version: 18.x
4747
cache: 'npm'
4848
- name: Install dependencies
4949
run: npm clean-install
@@ -68,7 +68,7 @@ jobs:
6868
strategy:
6969
matrix:
7070
os: [ubuntu-22.04]
71-
node: [16.x, 18.x, 20.x]
71+
node: [18.x, 20.x, 22.x]
7272
sqlserver: [2017, 2019, 2022]
7373
steps:
7474
- name: Checkout code
@@ -115,7 +115,7 @@ jobs:
115115
strategy:
116116
matrix:
117117
os: [windows-2019, windows-2022]
118-
node: [16.x, 18.x, 20.x]
118+
node: [18.x, 20.x, 22.x]
119119
sqlserver: [2008, 2012, 2014, 2016, 2017, 2019, 2022]
120120
# These sqlserver versions don't work on windows-2022 (at the moment)
121121
exclude:
@@ -153,8 +153,10 @@ jobs:
153153
- name: Run cli tests
154154
run: npm run test-cli
155155
- name: Install msnodesqlv8
156+
if: ${{ matrix.node != '22.x' }}
156157
run: npm install --no-save msnodesqlv8@^2
157158
- name: Run msnodesqlv8 tests
159+
if: ${{ matrix.node != '22.x' }}
158160
run: npm run test-msnodesqlv8
159161
release:
160162
name: Release

lib/datatypes.js

+1
Original file line numberDiff line numberDiff line change
@@ -160,6 +160,7 @@ module.exports.cast = (value, type, options) => {
160160
return `N'${value.replace(/'/g, '\'\'')}'`
161161

162162
case 'number':
163+
case 'bigint':
163164
return value
164165

165166
case 'boolean':

0 commit comments

Comments
 (0)