Skip to content

Add coverage_reporter_platform parameter#49

Merged
afinetooth merged 19 commits intomasterfrom
feat/add-coverage-reporter-platform
Nov 2, 2024
Merged

Add coverage_reporter_platform parameter#49
afinetooth merged 19 commits intomasterfrom
feat/add-coverage-reporter-platform

Conversation

@afinetooth
Copy link
Member

@afinetooth afinetooth commented Oct 24, 2024

Description

Add coverage_reporter_platform parameter with logic to download and use the appropriate architecture-specific binary of coverage-reporter.

To Do

  • Add coverage_reporter_platform parameter and logic to use architecture-specific binary of coverage-reporter.
  • Modify tests to test coverage_reporter_platform.

…iate architecture-specific binary of coverage-reporter. Add tests.
…ust test the last possible enum value, arm64.
…er to test unless we want to test fail_on_error.
…tempts, to deliver a helpful error message before a possible shell execution failure.
…tempts, to deliver a helpful error message before a possible shell execution failure.
…orrect value to match architecure in our script, and catch a change if it ever happens.
…erstand why we are sometimes getting this error.
…r version test, as it's verbose and redundant.
@afinetooth afinetooth merged commit 1f07055 into master Nov 2, 2024
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.

1 participant