feat: clear & disable timing after ready

Author: atian25

.github/workflows/ci.yml renamed to .github/workflows/nodejs.yml

@@ -1,27 +1,36 @@
 name: Continuous integration
-on: [push, pull_request]
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master, 1.x ]
+  schedule:
+    - cron: '0 2 * * *'
 jobs:
   Runner:
     runs-on: ${{ matrix.os }}
     strategy:
+      fail-fast: false
       matrix:
-        os: [ubuntu-latest, macOS-latest]
-        node-version: [8, 10, 12, 14]
+        os: [ ubuntu-latest, macOS-latest, windows-latest ]
+        node-version: [ 8, 10, 12, 14 ]
     steps:
     - name: Checkout Git Source
       uses: actions/checkout@master
+
     - name: Setup Node.js
       uses: actions/setup-node@v1
       with:
         node-version: ${{ matrix.node-version }}
+
     - name: Install Dependencies
       run: |
         npm install
+
     - name: Continuous integration
       run: npm run ci
+
     - name: Code Coverage
-      run: |
-        npm install codecov -g
-        codecov
-      env:
-        CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
+      uses: codecov/codecov-action@v1
+      with:
+        token: ${{ secrets.CODECOV_TOKEN }}

.travis.yml

@@ -11,15 +11,10 @@ node_js:
   - '12'
   - '14'
 before_install:
-  - npm install -g npminstall
+  - npm i npminstall -g
 install:
   - npminstall
-  - npminstall codecov
 script:
   - npm run ci
 after_script:
-  - codecov
-  - test $TRAVIS_NODE_VERSION = 8 && travis_wait scripts/doc_travis.sh
-env:
-  global:
-  - ENCRYPTION_LABEL: a62186272189
+  - npminstall codecov && codecov

appveyor.yml

@@ -411,6 +411,8 @@ class EggApplication extends EggCore {
       const rundir = this.config.rundir;
       const dumpFile = path.join(rundir, `${this.type}_timing_${process.pid}.json`);
       fs.writeFileSync(dumpFile, CircularJSON.stringify(json, null, 2));
+      this.timing.clear();
+      this.timing.disable();
     } catch (err) {
       this.coreLogger.warn(`dumpTiming error: ${err.message}`);
     }

lib/egg.js

@@ -25,7 +25,7 @@
     "delegates": "^1.0.0",
     "egg-cluster": "^1.23.0",
     "egg-cookies": "^2.3.0",
-    "egg-core": "^4.16.1",
+    "egg-core": "^4.18.0",
     "egg-development": "^2.4.2",
     "egg-i18n": "^2.0.0",
     "egg-jsonp": "^2.0.0",
@@ -115,5 +115,9 @@
   "engines": {
     "node": ">= 8.0.0"
   },
+  "ci": {
+    "type": "travis, github",
+    "version": "8, 10, 12, 14"
+  },
   "license": "MIT"
 }

package.json

@@ -145,6 +145,15 @@ describe('test/lib/egg.test.js', () => {
       assert(json[0].pid === process.pid);
     });
 
+    it('should disable & clear timing after ready', function* () {
+      const json = app.timing.toJSON();
+      assert(json.length === 0);
+      app.timing.start('a');
+      app.timing.end('a');
+      const json2 = app.timing.toJSON();
+      assert(json2.length === 0);
+    });
+
     it('should ignore error when dumpTiming', done => {
       mm(fs, 'writeFileSync', () => {
         throw new Error('mock error');