From c51bb942e8cd8ab2c11f051280b4d2ec76552b22 Mon Sep 17 00:00:00 2001 From: Ryan Zimmerman Date: Tue, 21 Sep 2021 14:14:22 -0400 Subject: [PATCH] Remove old broken CI tooling; now that we've switched to GH Actions Refs https://github.com/jprichardson/node-fs-extra/issues/874 --- .travis.yml | 25 ------------------------- README.md | 8 +++----- appveyor.yml | 27 --------------------------- package.json | 3 --- 4 files changed, 3 insertions(+), 60 deletions(-) delete mode 100644 .travis.yml delete mode 100644 appveyor.yml diff --git a/.travis.yml b/.travis.yml deleted file mode 100644 index 18757d03a..000000000 --- a/.travis.yml +++ /dev/null @@ -1,25 +0,0 @@ -language: node_js -os: - - linux - - osx -node_js: - - 12 - - 13 - - 14 - - 15 - - 16 -env: TEST_SUITE=unit -matrix: - exclude: - - os: linux - node_js: 12 - include: - - os: linux - node_js: 12 - env: TEST_SUITE=full-ci -script: npm run-script $TEST_SUITE -after_success: - - if [ $TEST_SUITE = full-ci ]; then npm run coveralls; fi -cache: - directories: - - node_modules diff --git a/README.md b/README.md index b4a5370b4..6ed8b6ae5 100644 --- a/README.md +++ b/README.md @@ -4,11 +4,9 @@ Node.js: fs-extra `fs-extra` adds file system methods that aren't included in the native `fs` module and adds promise support to the `fs` methods. It also uses [`graceful-fs`](https://github.com/isaacs/node-graceful-fs) to prevent `EMFILE` errors. It should be a drop in replacement for `fs`. [![npm Package](https://img.shields.io/npm/v/fs-extra.svg)](https://www.npmjs.org/package/fs-extra) -[![License](https://img.shields.io/npm/l/express.svg)](https://github.com/jprichardson/node-fs-extra/blob/master/LICENSE) -[![build status](https://img.shields.io/travis/jprichardson/node-fs-extra/master.svg)](http://travis-ci.org/jprichardson/node-fs-extra) -[![windows Build status](https://img.shields.io/appveyor/ci/jprichardson/node-fs-extra/master.svg?label=windows%20build)](https://ci.appveyor.com/project/jprichardson/node-fs-extra/branch/master) +[![License](https://img.shields.io/npm/l/fs-extra.svg)](https://github.com/jprichardson/node-fs-extra/blob/master/LICENSE) +[![build status](https://img.shields.io/github/workflow/status/jprichardson/node-fs-extra/Node.js%20CI/master)](https://github.com/jprichardson/node-fs-extra/actions/workflows/ci.yml?query=branch%3Amaster) [![downloads per month](http://img.shields.io/npm/dm/fs-extra.svg)](https://www.npmjs.org/package/fs-extra) -[![Coverage Status](https://img.shields.io/coveralls/github/jprichardson/node-fs-extra/master.svg)](https://coveralls.io/github/jprichardson/node-fs-extra) [![JavaScript Style Guide](https://img.shields.io/badge/code_style-standard-brightgreen.svg)](https://standardjs.com) Why? @@ -189,7 +187,7 @@ you're gonna have to get over it :) If `standard` is good enough for `npm`, it's What's needed? - First, take a look at existing issues. Those are probably going to be where the priority lies. - More tests for edge cases. Specifically on different platforms. There can never be enough tests. -- Improve test coverage. See coveralls output for more info. +- Improve test coverage. Note: If you make any big changes, **you should definitely file an issue for discussion first.** diff --git a/appveyor.yml b/appveyor.yml deleted file mode 100644 index a397dcba3..000000000 --- a/appveyor.yml +++ /dev/null @@ -1,27 +0,0 @@ -# Test against this version of Node.js -environment: - matrix: - # node.js - - nodejs_version: "12" - - nodejs_version: "14" -# Install scripts. (runs after repo cloning) -install: - # Get the latest stable version of Node.js or io.js - - ps: Install-Product node $env:nodejs_version - # install modules - - npm config set loglevel warn - - npm install --silent - -# Post-install test scripts. -test_script: - # Output useful info for debugging. - - node --version - - npm --version - # run only unit tests; travis does the linting - - npm run unit - -# Don't actually build. -build: off - -cache: - - node_modules diff --git a/package.json b/package.json index b25936cae..8a89b01f5 100644 --- a/package.json +++ b/package.json @@ -43,7 +43,6 @@ }, "devDependencies": { "at-least-node": "^1.0.0", - "coveralls": "^3.0.0", "klaw": "^2.1.1", "klaw-sync": "^3.0.2", "minimist": "^1.1.1", @@ -59,9 +58,7 @@ "!lib/**/__tests__/" ], "scripts": { - "full-ci": "npm run lint && npm run coverage", "coverage": "nyc -r lcovonly npm run unit", - "coveralls": "coveralls < coverage/lcov.info", "lint": "standard", "test-find": "find ./lib/**/__tests__ -name *.test.js | xargs mocha", "test": "npm run lint && npm run unit",