Skip to content

Commit

Permalink
Merge pull request #750 from ain/feature/nodev22win-support
Browse files Browse the repository at this point in the history
Remove Windows test exclusion for Node.js v22+
Resolve #786
  • Loading branch information
ain authored Oct 17, 2024
2 parents d858c17 + 4b4f9d4 commit e96d1a9
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 14 deletions.
11 changes: 1 addition & 10 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,23 +9,14 @@ jobs:
matrix:
os: [ubuntu-latest, windows-latest]
node: [17, 18, 19, 20, 21, 22, 23]
exclude:
- os: windows-latest
node: 22
- os: windows-latest
node: 23
steps:
- uses: actions/checkout@v4
- name: Use Node.js ${{ matrix.node }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node }}
- run: npm install
- if: ${{ matrix.os == 'windows-latest' }}
run: |
npm run eslint -- --rule 'linebreak-style: [2, windows]'
- if: ${{ matrix.os == 'ubuntu-latest' }}
run: npm run eslint
- run: npm run eslint
- run: npm test
- run: npm run build
- name: Coveralls Parallel
Expand Down
4 changes: 0 additions & 4 deletions eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,10 +5,6 @@ module.exports = [
"error",
2
],
"linebreak-style": [
"error",
"unix"
],
"quotes": [
"error",
"single"
Expand Down

0 comments on commit e96d1a9

Please sign in to comment.