Skip to content

Commit

Permalink
comment out windows test -- can't get it to work
Browse files Browse the repository at this point in the history
  • Loading branch information
talmobi committed Sep 26, 2024
1 parent 7db94e6 commit be7c2b2
Showing 1 changed file with 37 additions and 34 deletions.
71 changes: 37 additions & 34 deletions .github/workflows/windows-node.js.yml
Original file line number Diff line number Diff line change
@@ -1,34 +1,37 @@
name: windows

on:
push:
branches: [ "master" ]
pull_request:
branches: [ "master" ]

jobs:
build:

runs-on: windows-latest

strategy:
matrix:
node-version: [20.x]
# See supported Node.js release schedule at https://nodejs.org/en/about/releases/

steps:
- uses: actions/checkout@v4

- name: Use Node.js ${{ matrix.node-version }}
uses: actions/setup-node@v1
with:
node-version: ${{ matrix.node-version }}
cache: 'npm'

- name: Workaround to create TTY
shell: pwsh
run: |
npm.cmd install
npm.cmd run build --if-present
winpty npm.cmd test
# not sure if it's possible to test this fully on windows that doesn't have
# similar TTY support?
##########################
# name: windows
#
# on:
# push:
# branches: [ "master" ]
# pull_request:
# branches: [ "master" ]
#
# jobs:
# build:
#
# runs-on: windows-latest
#
# strategy:
# matrix:
# node-version: [20.x]
# # See supported Node.js release schedule at https://nodejs.org/en/about/releases/
#
# steps:
# - uses: actions/checkout@v4
#
# - name: Use Node.js ${{ matrix.node-version }}
# uses: actions/setup-node@v1
# with:
# node-version: ${{ matrix.node-version }}
# cache: 'npm'
#
# - name: Workaround to create TTY
# shell: pwsh
# run: |
# npm.cmd install
# npm.cmd run build --if-present
# winpty npm.cmd test
#

0 comments on commit be7c2b2

Please sign in to comment.