Merge pull request #191 from tiobe/33012-retry_proxy

Fixed the issue regarding using proxies with the previous implementation
tiobe · Oct 26, 2023 · 0ed5e17 · 0ed5e17
2 parents ccf8ba3 + a88d555
commit 0ed5e17
Show file tree

Hide file tree

Showing 38 changed files with 74,540 additions and 17,545 deletions.
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -13,4 +13,6 @@ jobs:
       - name: Jest
         uses: actions/setup-node@v3
       - run: npm ci
-      - run: npm run test-coverage
+      - run: npm run test:coverage
+        env:
+          INPUT_GITHUBTOKEN: ${{ secrets.GITHUB_TOKEN }}
diff --git a/.github/workflows/tics-analyzer.yml b/.github/workflows/tics-analyzer.yml
@@ -13,7 +13,9 @@ jobs:
       - name: Setup Node.js environment
         uses: actions/setup-node@v3
       - run: npm ci
-      - run: npm run test-coverage
+      - run: npm run test:coverage
+        env:
+          INPUT_GITHUBTOKEN: ${{ secrets.GITHUB_TOKEN }}
       - name: Install and run TICSQSERVER
         env:
           TICSAUTHTOKEN: ${{ secrets.EBOIT_ANALYZER_TOKEN }}

diff --git a/.github/workflows/tics-client.yml b/.github/workflows/tics-client.yml
@@ -10,6 +10,9 @@ jobs:
       - name: Setup Node.js environment
         uses: actions/setup-node@v3
       - run: npm ci
+      - run: npm run test:coverage
+        env:
+          INPUT_GITHUBTOKEN: ${{ secrets.GITHUB_TOKEN }}
       - name: TICS GitHub Action
         uses: tiobe/tics-github-action@v2
         with:

diff --git a/.npmrc b/.npmrc
@@ -0,0 +1,2 @@
+@tiobe:registry=https://artifacts.tiobe.com/repository/npm/
+registry=https://registry.npmjs.org
diff --git a/README.md b/README.md
@@ -83,6 +83,7 @@ The following inputs are available for this action:
 | `postAnnotations`      | Show the latest TICS annotations directly in the GitHub Pull Request review.                                                                                                                                                                                                             | false    |
 | `postToConversation`   | Post the summary to the conversation page of the pull request. Options are `true` (default) or `false`.                                                                                                                                                                                  | false    |
 | `pullRequestApproval`  | Set the plugin to approve or deny a pull request, by default this is false. Options are `true` or `false`. Note that once a run that added a reviewer has been completed, this reviewer cannot be deleted from that pull request. (Always the case on versions between 2.0.0 and 2.5.0). | false    |
+| `retryCodes`           | Status codes to retry api calls for. The default codes will be overwritten if this option is set.                                                                                                                                                                                        | false    |
 | `secretsFilter`        | Comma-seperated list of extra secrets to mask in the console output.                                                                                                                                                                                                                     | false    |
 | `ticsAuthToken`        | Authentication token to authorize the plugin when it connects to the TICS Viewer.                                                                                                                                                                                                        | false    |
 | `tmpDir`               | Location to store debug information.                                                                                                                                                                                                                                                     | false    |
@@ -95,6 +96,7 @@ The following inputs are available for this action:
 - To package the build to run run `npm run package`.
 - To combine the last two steps run `npm run all`.
 - There is Prettier auto-formatting available, run `npm run format` or enable format on save to automate the formatting.
+- In order to run the tests the environment variable `INPUT_GITHUBTOKEN` needs to be set with a valid `GITHUB_TOKEN`
 
 ## Git hooks
 

diff --git a/action.yaml b/action.yaml
@@ -74,6 +74,9 @@ inputs:
     description: Set the plugin to approve or deny a pull request, by default this is false. Options are `true` or `false`. Note that once a run that added a reviewer has been completed, this reviewer cannot be deleted from that pull request. (Always the case on versions between 2.0.0 and 2.5.0).
     required: false
     default: false
+  retryCodes:
+    description: Status codes to retry api calls for. The default codes will be overwritten if this option is set.
+    required: false
   secretsFilter:
     description: Comma-seperated list of extra secrets to mask in the console output.
     required: false
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		@tiobe:registry=https://artifacts.tiobe.com/repository/npm/
		registry=https://registry.npmjs.org