Skip to content

Update action.yml to support --upload-batch-size parameter#69

Merged
pedrocodacy merged 2 commits intomasterfrom
fix/add-support-for-upload-batch-size-CY-5756
Mar 4, 2022
Merged

Update action.yml to support --upload-batch-size parameter#69
pedrocodacy merged 2 commits intomasterfrom
fix/add-support-for-upload-batch-size-CY-5756

Conversation

@pedrocodacy
Copy link
Copy Markdown
Contributor

did not know we were maintaining the version of the CLI manually here, any reasons fo us to not be using latest? (not changing that for now)

@prcr
Copy link
Copy Markdown
Contributor

prcr commented Mar 3, 2022

If it helps the subsequent discussion, this was the pull request where we started to pin the Codacy Analisis CLI version used by the GitHub Action: #11

lucassklp
lucassklp previously approved these changes Mar 3, 2022
Comment thread action.yml Outdated
description: "Upload analysis results to Codacy"
upload-batch-size:
required: false
description: "Size of result batches to be uploaded to Codacy"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should mention the unit for the batch size. I'm guessing it's bytes? The Codacy Analysis CLI doesn't mention it either.

Comment thread action.yml Outdated
@pedrocodacy pedrocodacy dismissed stale reviews from lucassklp and stefanvacareanu7 via 44a0029 March 4, 2022 12:00
@pedrocodacy pedrocodacy merged commit 166037e into master Mar 4, 2022
@pedrocodacy pedrocodacy deleted the fix/add-support-for-upload-batch-size-CY-5756 branch March 4, 2022 12:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants