Skip to content

Commit 3589f23

Browse files
committed
fix(build): revert node: prefix in copy-artifacts
The prebuildify Docker images for linuxglibc use Node.js 12 which doesn't support the node: protocol prefix in require calls. This was causing CI failures during the build step.
1 parent cca7772 commit 3589f23

File tree

2 files changed

+16
-3
lines changed

2 files changed

+16
-3
lines changed

eslint.config.js

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,6 +45,19 @@ module.exports = [
4545
},
4646
},
4747
},
48+
{
49+
// This script runs inside the prebuildify Docker container which uses Node.js 12
50+
files: ['scripts/copy-artifacts.js'],
51+
languageOptions: {
52+
ecmaVersion: 2019,
53+
},
54+
settings: {
55+
node: { version: '>=12.0.0' },
56+
},
57+
rules: {
58+
'unicorn/prefer-node-protocol': 'off',
59+
},
60+
},
4861
{
4962
ignores: ['build/', 'target/', 'prebuilds/'],
5063
},

scripts/copy-artifacts.js

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
'use strict'
22

3-
const fs = require('node:fs')
4-
const path = require('node:path')
5-
const readline = require('node:readline')
3+
const fs = require('fs')
4+
const path = require('path')
5+
const readline = require('readline')
66

77
const rootPath = path.resolve(path.join(__dirname, '..'))
88
const cratesPath = path.join(rootPath, 'crates')

0 commit comments

Comments
 (0)