Skip to content

Commit

Permalink
Revert "chore(deps): bump @azure/identity and tedious"
Browse files Browse the repository at this point in the history
  • Loading branch information
dhensby committed Jun 18, 2024
1 parent a655df2 commit ade3f8a
Show file tree
Hide file tree
Showing 4 changed files with 193 additions and 257 deletions.
10 changes: 4 additions & 6 deletions .github/workflows/nodejs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@ jobs:
- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: 18.x
node-version: 16.x
cache: 'npm'
- name: Install dependencies
run: npm clean-install
Expand All @@ -43,7 +43,7 @@ jobs:
- name: Setup Node.js
uses: actions/setup-node@v4
with:
node-version: 18.x
node-version: 16.x
cache: 'npm'
- name: Install dependencies
run: npm clean-install
Expand All @@ -68,7 +68,7 @@ jobs:
strategy:
matrix:
os: [ubuntu-22.04]
node: [18.x, 20.x, 22.x]
node: [16.x, 18.x, 20.x]
sqlserver: [2017, 2019, 2022]
steps:
- name: Checkout code
Expand Down Expand Up @@ -115,7 +115,7 @@ jobs:
strategy:
matrix:
os: [windows-2019, windows-2022]
node: [18.x, 20.x, 22.x]
node: [16.x, 18.x, 20.x]
sqlserver: [2008, 2012, 2014, 2016, 2017, 2019, 2022]
# These sqlserver versions don't work on windows-2022 (at the moment)
exclude:
Expand Down Expand Up @@ -153,10 +153,8 @@ jobs:
- name: Run cli tests
run: npm run test-cli
- name: Install msnodesqlv8
if: ${{ matrix.node != '22.x' }}
run: npm install --no-save msnodesqlv8@^2
- name: Run msnodesqlv8 tests
if: ${{ matrix.node != '22.x' }}
run: npm run test-msnodesqlv8
release:
name: Release
Expand Down
1 change: 0 additions & 1 deletion lib/datatypes.js
Original file line number Diff line number Diff line change
Expand Up @@ -160,7 +160,6 @@ module.exports.cast = (value, type, options) => {
return `N'${value.replace(/'/g, '\'\'')}'`

case 'number':
case 'bigint':
return value

case 'boolean':
Expand Down
Loading

0 comments on commit ade3f8a

Please sign in to comment.