Fail CI if true

fix: Build with ts-4.3.2

.github/dependabot.yml

@@ -0,0 +1,7 @@
+version: 2
+updates:
+- package-ecosystem: npm
+  directory: "/"
+  schedule:
+    interval: daily
+  open-pull-requests-limit: 10

.github/workflows/main.yml

@@ -15,6 +15,7 @@ jobs:
       - name: Upload coverage to Codecov (demo)
         uses: ./
         with:
+          fail_ci_if_error: false
           files: ./coverage/calculator/coverage-final.json,./coverage/coverage-test/coverage-final.json
           file: ./coverage/coverage-final.json
           flags: demo
@@ -22,6 +23,7 @@ jobs:
       - name: Upload coverage to Codecov (script)
         uses: ./
         with:
+          fail_ci_if_error: true
           files: ./coverage/script/coverage-final.json
           flags: script
           name: codecov-script

CHANGELOG.md

@@ -1,43 +1,98 @@
-### 1.3.0
+## 1.5.2
+### Fixes
+- # fix: Import version properly as string not object
 
-#### Features
+## 1.5.1
+### Fixes
+- #320 doc: add github actions badge
+- #336 Update bash uploader to 1.0.3
+- #339 fix: Add action version
+
+### Dependencies
+- #302 Bump @typescript-eslint/eslint-plugin from 4.22.0 to 4.22.1
+- #303 Bump @typescript-eslint/parser from 4.22.0 to 4.22.1
+- #304 Bump ts-jest from 26.5.5 to 26.5.6
+- #309 Bump lodash from 4.17.19 to 4.17.21
+- #310 Bump hosted-git-info from 2.8.8 to 2.8.9
+- #311 Bump @actions/github from 4.0.0 to 5.0.0
+- #314 Bump eslint from 7.25.0 to 7.27.0
+- #315 Bump @actions/core from 1.2.7 to 1.3.0
+- #316 Bump @typescript-eslint/parser from 4.22.1 to 4.25.0
+- #317 Bump @typescript-eslint/eslint-plugin from 4.22.1 to 4.25.0
+- #319 Bump jest-junit from 12.0.0 to 12.1.0
+- #321 Bump typescript from 4.2.4 to 4.3.2
+- #323 Bump ws from 7.3.1 to 7.4.6
+- #331 Bump eslint from 7.27.0 to 7.28.0
+- #332 Bump @actions/exec from 1.0.4 to 1.1.0
+- #333 Bump @typescript-eslint/parser from 4.25.0 to 4.26.1
+- #334 Bump @typescript-eslint/eslint-plugin from 4.25.0 to 4.26.1
+- #335 Bump @actions/core from 1.3.0 to 1.4.0
+- #337 Bump glob-parent from 5.1.1 to 5.1.2
+
+## 1.5.0
+### Features
+- #299 Pull Codecov bash script into the action
+
+### Dependencies
+- #271 Bump typescript from 4.2.3 to 4.2.4
+- #277 Bump @typescript-eslint/eslint-plugin from 4.16.1 to 4.22.0
+- #278 Bump @typescript-eslint/parser from 4.20.0 to 4.22.0
+- #279 Bump @actions/core from 1.2.6 to 1.2.7
+- #292 Bump ts-jest from 26.5.3 to 26.5.5
+- #293 Bump eslint from 7.21.0 to 7.25.0
+- #297 Bump @types/jest from 26.0.20 to 26.0.23
+- #298 Upgrade to GitHub-native Dependabot
+
+## 1.4.1
+### Fixes
+- #287 Update VERSION regex to restrict on digits and dot and move checksums into script
+
+## 1.4.0
+### Features
+- #282 Add checksum verification of bash script
+
+## 1.3.2
+### Fixes
+- #264 Overwrites pr number for pull_request_target events
+
+## 1.3.1
+### Fixes
+- #253 Add `network_filter` to action manifest
+
+## 1.3.0
+### Features
 - #252 Add "network_filter" input
 
-### 1.2.2
+## 1.2.2
-
+### Fixes
-#### Fixes
 - #241 pass root_dir using proper bash arg
 - #244 Overwrite the commit on pull_request* events
 
-### 1.2.1
+## 1.2.1
-
+### Fixes
-#### Fixes
 - #196 Add parameters to the action.yml
 
-### 1.2.0
+## 1.2.0
-
+### Features
-#### Features
 - #193 Add all the bash params
 
-#### Fixes
+### Fixes
 - #193 Fixes issue with working-directory
 
-### 1.1.1
+## 1.1.1
-
+### Fixes
-#### Fixes
 - #184 Add automations ensure proper builds and deployments
 - #184 Fixes verbose flag
 
-### 1.1.0
+## 1.1.0
-
+### Features
-#### Features
 - #110 Add "working-directory:" input
 - #174 Support Xcode specificed parameters
 
-#### Fixes
+### Fixes
 - #172 File is saved as text
 
-#### Dependencies and Misc
+### Dependencies and Misc
 - #166 Bump requestretry from 4.1.1 to 4.1.2
 - #169 Bump typescript from 4.0.5 to 4.1.2
 - #178 Bump @types/jest from 26.0.15 to 26.0.19

README.md

@@ -2,6 +2,7 @@
 
 [![GitHub Marketplace](https://img.shields.io/badge/Marketplace-v1-undefined.svg?logo=github&logoColor=white&style=flat)](https://github.com/marketplace/actions/codecov)
 [![FOSSA Status](https://app.fossa.com/api/projects/git%2Bgithub.com%2Fcodecov%2Fcodecov-action.svg?type=shield)](https://app.fossa.com/projects/git%2Bgithub.com%2Fcodecov%2Fcodecov-action?ref=badge_shield)
+[![Workflow for Codecov Action](https://github.com/codecov/codecov-action/actions/workflows/main.yml/badge.svg)](https://github.com/codecov/codecov-action/actions/workflows/main.yml)
 ### Easily upload coverage reports to Codecov from GitHub Actions
 
 >The latest release of this Action adds support for tokenless uploads from GitHub Actions!

action.yml

@@ -59,6 +59,9 @@ inputs:
   name:
     description: 'User defined upload name. Visible in Codecov UI'
     required: false
+  network_filter:
+    description: 'Used to restrict the set of git/hg files that can be matched with filenames in the coverage report. This is useful for monorepos or other setups where a full filepath may not be specified in the coverage report, and that shortened filepath may appear multiple times in a directory structure (e.g. __init__.py)'
+    required: false
   override_branch:
     description: 'Specify the branch name'
     required: false

hooks/pre-commit

@@ -1,8 +1,10 @@
 #!/usr/bin/env bash
 
-npm i --package-lock-only
+set -e
+
+npm install
 npm run lint --fix
 npm run build
 git add src/
-git add dist/index.js
+git add dist/
 git add package-lock.json

package.json

@@ -1,6 +1,6 @@
 {
   "name": "codecov-action",
-  "version": "1.3.0",
+  "version": "1.5.2",
   "description": "Upload coverage reports to Codecov from GitHub Actions",
   "main": "index.js",
   "scripts": {
@@ -23,24 +23,24 @@
   },
   "homepage": "https://github.com/codecov/codecov-action#readme",
   "dependencies": {
-    "@actions/core": "^1.2.6",
+    "@actions/core": "^1.4.0",
-    "@actions/exec": "^1.0.4",
+    "@actions/exec": "^1.1.0",
-    "@actions/github": "^4.0.0",
+    "@actions/github": "^5.0.0",
-    "@types/jest": "^26.0.20",
-    "@zeit/ncc": "^0.22.3",
     "fs": "0.0.1-security",
-    "jest": "^26.6.3",
+    "request": "^2.88.2"
-    "jest-junit": "^12.0.0",
-    "request": "^2.88.2",
-    "requestretry": "^5.0.0",
-    "ts-jest": "^26.5.3",
-    "typescript": "^4.2.3",
-    "yarn": "^1.22.10"
   },
   "devDependencies": {
-    "@typescript-eslint/eslint-plugin": "^4.16.1",
+    "@types/jest": "^26.0.23",
-    "@typescript-eslint/parser": "^4.16.1",
+    "@types/node": "^14.17.3",
-    "eslint": "^7.21.0",
+    "@typescript-eslint/eslint-plugin": "^4.26.1",
-    "eslint-config-google": "^0.14.0"
+    "@typescript-eslint/parser": "^4.26.1",
+    "@zeit/ncc": "^0.22.3",
+    "eslint": "^7.28.0",
+    "eslint-config-google": "^0.14.0",
+    "jest": "^26.6.3",
+    "jest-junit": "^12.2.0",
+    "ts-jest": "^26.5.6",
+    "typescript": "^4.3.2",
+    "yarn": "^1.22.10"
   }
 }

src/buildExec.test.ts

@@ -1,5 +1,8 @@
 import buildExec from './buildExec';
 const github = require('@actions/github');
+
+import VERSION from './version';
+
 const context = github.context;
 
 test('no arguments', () => {
@@ -12,7 +15,7 @@ test('no arguments', () => {
     '-F',
     '',
     '-Q',
-    'github-action',
+    `github-action-${VERSION}`,
   ];
   if (context.eventName == 'pull_request') {
     args.push('-C', `${context.payload.pull_request.head.sha}`);
@@ -70,7 +73,7 @@ test('all arguments', () => {
     '-F',
     'test',
     '-Q',
-    'github-action',
+    `github-action-${VERSION}`,
     '-c',
     '-N',
     '83231650328f11695dfb754ca0f540516f188d27',

src/buildExec.ts

@@ -1,6 +1,8 @@
 const core = require('@actions/core');
 const github = require('@actions/github');
 
+import VERSION from './version';
+
 const context = github.context;
 
 const isTrue = (variable) => {
@@ -51,7 +53,14 @@ const buildExec = () => {
     workingDir + '/codecov.sh' : 'codecov.sh';
 
   const execArgs = [filepath];
-  execArgs.push( '-n', `${name}`, '-F', `${flags}`, '-Q', 'github-action' );
+  execArgs.push(
+      '-n',
+      `${name}`,
+      '-F',
+      `${flags}`,
+      '-Q',
+      `github-action-${VERSION}`,
+  );
 
   const options:any = {};
   options.env = Object.assign(process.env, {
@@ -143,6 +152,10 @@ const buildExec = () => {
   }
   if (overridePr) {
     execArgs.push('-P', `${overridePr}`);
+  } else if (
+    `${context.eventName}` == 'pull_request_target'
+  ) {
+    execArgs.push('-P', `${context.payload.number}`);
   }
   if (overrideTag) {
     execArgs.push('-T', `${overrideTag}`);

src/index.ts

@@ -2,63 +2,47 @@ const core = require('@actions/core');
 const exec = require('@actions/exec');
 
 const fs = require('fs');
-const request = require('requestretry');
 
 import buildExec from './buildExec';
 
+const codecovScript = fs.readFileSync(__dirname + '/codecov');
+
 let failCi;
 try {
-  request({
+  const {execArgs, options, filepath, failCi} = buildExec();
-    json: false,
-    maxAttempts: 10,
-    timeout: 3000,
-    url: 'https://codecov.io/bash',
-  }, (error, response, body) => {
-    const {execArgs, options, filepath, failCi} = buildExec();
 
-    try {
+  fs.writeFile(filepath, codecovScript, (err) => {
-      if (error && failCi) {
+    if (err && failCi) {
-        throw error;
+      throw err;
-      } else if (error) {
+    } else if (err) {
-        core.warning(`Codecov warning: ${error.message}`);
+      core.warning(`Codecov warning: ${err.message}`);
-      }
+    }
 
-      fs.writeFile(filepath, body, (err) => {
+    exec.exec('bash', execArgs, options)
+        .catch((err) => {
+          console.log(err);
+          if (failCi) {
+            core.setFailed(
+                `Codecov failed with the following error: ${err.message}`,
+            );
+          } else {
+            core.warning(`Codecov warning: ${err.message}`);
+          }
+        })
+        .then((status) => {
+          console.log(status);
+          unlinkFile();
+        });
+
+    const unlinkFile = () => {
+      fs.unlink(filepath, (err) => {
         if (err && failCi) {
           throw err;
         } else if (err) {
           core.warning(`Codecov warning: ${err.message}`);
         }
-
-        exec.exec('bash', execArgs, options)
-            .catch((err) => {
-              if (failCi) {
-                core.setFailed(
-                    `Codecov failed with the following error: ${err.message}`,
-                );
-              } else {
-                core.warning(`Codecov warning: ${err.message}`);
-              }
-            })
-            .then(() => {
-              unlinkFile();
-            });
-
-        const unlinkFile = () => {
-          fs.unlink(filepath, (err) => {
-            if (err && failCi) {
-              throw err;
-            } else if (err) {
-              core.warning(`Codecov warning: ${err.message}`);
-            }
-          });
-        };
       });
-    } catch (error) {
+    };
-      core.setFailed(
-          `Codecov failed with the following error: ${error.message}`,
-      );
-    }
   });
 } catch (error) {
   if (failCi) {

src/version.ts

@@ -0,0 +1,3 @@
+const VERSION = 'v1.5.2';
+
+export default VERSION;