Skip to content

Generate test build artifacts #179

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 31, 2021
Merged

Generate test build artifacts #179

merged 2 commits into from
May 31, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented May 30, 2021

Providing pre-built binaries for every pull request or commit that changes relevant files enables any interested party to
participate in the review or beta testing process without imposing the requirement to set up the build toolchain on their
local machine.

This will also avoid any chance of differences between the tester's environment and the release system environment
introducing spurious results.

per1234 added 2 commits May 30, 2021 05:15
Providing pre-built binaries for every pull request or commit that changes relevant files enables any interested party to
participate in the review or beta testing process without imposing the requirement to set up the build toolchain on their
local machine.

This will also avoid any chance of differences between the tester's environment and the release system environment
introducing spurious results.
This build step became superfluous with the reconfiguration of the integration test task to do the necessary build.
@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels May 30, 2021
@per1234 per1234 requested review from silvanocerza and umbynos May 30, 2021 12:43
@codecov-commenter
Copy link

Codecov Report

Merging #179 (1eedcc6) into main (182dbba) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #179   +/-   ##
=======================================
  Coverage   84.36%   84.36%           
=======================================
  Files          43       43           
  Lines        2986     2986           
=======================================
  Hits         2519     2519           
  Misses        374      374           
  Partials       93       93           
Flag Coverage Δ
unit 84.36% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 182dbba...1eedcc6. Read the comment docs.

@per1234 per1234 merged commit 45b32ff into arduino:main May 31, 2021
@per1234 per1234 deleted the test-artifact branch May 31, 2021 08:28
@per1234 per1234 self-assigned this Nov 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: infrastructure Related to project infrastructure type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants