Skip to content

Finish sync of build infrastructure with "template" assets #242

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 3 commits into from
Aug 6, 2021
Merged

Finish sync of build infrastructure with "template" assets #242

merged 3 commits into from
Aug 6, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 6, 2021

Some additional changes missed on the first pass (#222), which are required for a full sync with the standardized "template" project build assets.

@per1234 per1234 added the topic: infrastructure Related to project infrastructure label Aug 6, 2021
@per1234 per1234 requested review from silvanocerza and umbynos August 6, 2021 01:11
per1234 added 3 commits August 6, 2021 02:32
The project specific filename makes the file less suitable to be used as a "template" for application to any project.
Since this file is not used for anything, there is no need for it to have any special name.
Tester builds are intended to facilitate pull request review and beta testing. For the sake of efficiency, the paths
filter should be configured to skip tester build generation on changes that have no effect on the binary, but also so
that they are always generated on any modification to a relevant file. Since the `DistTasks.yml` file is used to build
the application, it is very much relevant.
@per1234 per1234 merged commit f5072eb into arduino:main Aug 6, 2021
@per1234 per1234 deleted the sync-release-infrastructure branch August 6, 2021 09:49
@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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants