Skip to content

Commit

Permalink
CI fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
kjvalencik committed Feb 13, 2022
1 parent 8b03202 commit 53944f1
Show file tree
Hide file tree
Showing 7 changed files with 21 additions and 121 deletions.
41 changes: 0 additions & 41 deletions .github/workflows/lint.yml

This file was deleted.

36 changes: 0 additions & 36 deletions .github/workflows/linux.yml

This file was deleted.

35 changes: 0 additions & 35 deletions .github/workflows/macos.yml

This file was deleted.

25 changes: 16 additions & 9 deletions .github/workflows/windows.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,10 @@ jobs:

strategy:
matrix:
node-version: [12.x, 14.x, 16.x]
rust-toolchain: [stable, beta, nightly]
# node-version: [12.x, 14.x, 16.x]
# rust-toolchain: [stable, beta, nightly]
node-version: [ 16.x ]
rust-toolchain: [ stable ]

steps:
- uses: actions/checkout@v2
Expand All @@ -29,18 +31,23 @@ jobs:
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node-version }}
- name: Use npm v6
if: ${{ matrix.node-version == '16.x' }}
run: npm install -g npm@6
- name: Install libclang
uses: KyleMayes/install-llvm-action@01144dc97b1e2693196c3056414a44f15180648b
with:
version: "10"
directory: ${{ runner.temp }}/llvm
- run: npm config set msvs_version 2019
- name: Use npm v8
# Windows bug in npm 8.4
# https://github.com/npm/cli/issues/4341
run: npm i npm@~8.3
working-directory: ${{ runner.temp }}
- name: Update header files
run: npm i -g node-gyp@latest && node-gyp install
- name: Configure MSVS version
run: npm config set msvs_version 2022
- name: npm install workspace
run: npm install
- name: run tests
run: npm test
run: ${{ runner.temp }}\node_modules\.bin\npm.cmd install
env:
LIBCLANG_PATH: ${{ runner.temp }}/llvm/bin
- name: run tests
run: ${{ runner.temp }}\node_modules\.bin\npm.cmd test
2 changes: 2 additions & 0 deletions .prettierignore
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@
.git/
cli/lib/
test/cli/lib/
node_modules/
create-neon/dist/
target/
artifacts.json
2 changes: 2 additions & 0 deletions crates/neon-build/src/napi/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ fn test_absolute_output_file_defaults() {
}

#[test]
#[cfg_attr(windows, ignore)]
fn test_absolute_output_file_absolute_file() {
let expected = PathBuf::from("/tmp/hello.node");
let actual = Setup::options()
Expand All @@ -145,6 +146,7 @@ fn test_absolute_output_file_absolute_file() {
}

#[test]
#[cfg_attr(windows, ignore)]
fn test_absolute_output_file_absolute_dir() {
let expected = PathBuf::from("/tmp/index.node");
let actual = Setup::options().output_dir("/tmp").absolute_output_file();
Expand Down
1 change: 1 addition & 0 deletions crates/neon-sys/native/binding.gyp
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
'Debug': {
'msvs_settings': {
'VCCLCompilerTool': {
'WholeProgramOptimization': 'false',
'RuntimeLibrary': '0',
'UndefinePreprocessorDefinitions': ['DEBUG', '_DEBUG'],
'PreprocessorDefinitions': ['NDEBUG']
Expand Down

0 comments on commit 53944f1

Please sign in to comment.