Skip to content
This repository was archived by the owner on Jul 23, 2025. It is now read-only.

Commit abc8099

Browse files
committed
Do not add a batch size default value in the action as cc-test-reporter tool can change this in the future
1 parent a611b9b commit abc8099

File tree

3 files changed

+5
-8
lines changed

3 files changed

+5
-8
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ This action requires that you set the [`CC_TEST_REPORTER_ID`](https://docs.codec
2121
| `prefix` | `undefined` | See [`--prefix`](https://docs.codeclimate.com/docs/configuring-test-coverage) |
2222
| `verifyDownload` | `true` | Verifies the downloaded Code Climate reporter binary's checksum and GPG signature. See [Verifying binaries](https://github.com/codeclimate/test-reporter#verifying-binaries) |
2323
| `verifyEnvironment` | `true` | Verifies the current runtime environment (operating system and CPU architecture) is supported by the Code Climate reporter. See [list of supported platforms](https://github.com/codeclimate/test-reporter#binaries) |
24-
| `batchSize` | `500` | Batch size for source files used by upload-coverage command. |
24+
| `batchSize` | | Batch size for source files (cc-test-reporter upload-coverage uses 500 by default) |
2525

2626
> **Note**
2727
> If you are a Ruby developer using [SimpleCov](https://github.com/simplecov-ruby/simplecov), other users have recommended installing an additional gem – `gem "simplecov_json_formatter"` – this gem fixes `json` error from the default `coverage/.resultset.json` output from SimpleCov.

action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@ inputs:
3535
default: 'true'
3636
batchSize:
3737
required: false
38-
description: 'Batch size for source files to be sent to Code Climate in upload coverage step'
39-
default: '500'
38+
description: 'Batch size for source files (cc-test-reporter upload-coverage uses 500 by default)'
39+
default: ''
4040
runs:
4141
using: 'node20'
4242
main: 'lib/main.mjs'

src/main.ts

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -212,7 +212,7 @@ export async function run({
212212
coveragePrefix,
213213
verifyDownload = DEFAULT_VERIFY_DOWNLOAD,
214214
verifyEnvironment = DEFAULT_VERIFY_ENVIRONMENT,
215-
batchSize = DEFAULT_BATCH_SIZE,
215+
batchSize,
216216
}: ActionArguments = {}): Promise<void> {
217217
let lastExitCode = 1;
218218
if (verifyEnvironment === 'true') {
@@ -439,10 +439,7 @@ if (isThisFileBeingRunViaCLI) {
439439
'verifyEnvironment',
440440
DEFAULT_VERIFY_ENVIRONMENT,
441441
);
442-
const batchSize = getOptionalString(
443-
'batchSize',
444-
DEFAULT_BATCH_SIZE,
445-
);
442+
const batchSize = getOptionalString('batchSize');
446443

447444
try {
448445
run({

0 commit comments

Comments
 (0)