Skip to content

Commit ee19dd3

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 f613e49 commit ee19dd3

2 files changed

Lines changed: 17 additions & 3 deletions

File tree

eslint.config.js

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,20 @@ module.exports = [
5151
},
5252
},
5353
},
54+
{
55+
// This script runs inside the prebuildify Docker container which uses Node.js 12
56+
files: ['scripts/copy-artifacts.js'],
57+
languageOptions: {
58+
ecmaVersion: 2019,
59+
},
60+
settings: {
61+
// Used by `eslint-plugin-n` to determine the minimum version of Node.js to support.
62+
node: { version: '>=12.0.0' },
63+
},
64+
rules: {
65+
'unicorn/prefer-node-protocol': 'off',
66+
},
67+
},
5468
{
5569
ignores: ['build/', 'target/', 'prebuilds/'],
5670
},

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)