Skip to content
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

Only download required artifact in notarization job of release workflows #531

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Nov 6, 2024

GitHub Workflows are used to automatically generate and publish production and nightly releases of the project. This is done for a range of host architectures, including macOS. The macOS builds are then put through a notarization process in a dedicated workflow job.

GitHub Actions workflow artifacts are used to transfer the generated files between sequential jobs in the workflow. The builds are transferred between jobs by GitHub Actions workflow artifacts, one for each host architecture.

Previously, the "notarize-macos" job matrix that performs the notarization unnecessarily downloaded all the build artifacts, even though each job only requires the relevant macOS artifact. This is inefficient.

The better approach is to configure the "notarize-macos" jobs to only download the artifact they require.

GitHub Workflows are used to automatically generate and publish production and nightly releases of the project. This is
done for a range of host architectures, including macOS. The macOS builds are then put through a notarization process in
a dedicated workflow job.

GitHub Actions workflow artifacts are used to transfer the generated files between sequential jobs in the workflow. The
builds are transferred between jobs by GitHub Actions workflow artifacts, one for each host architecture.

Previously, the "notarize-macos" job matrix that performs the notarization unnecessarily downloaded all the build
artifacts, even though each job only requires the relevant macOS artifact. This is inefficient.

The better approach is to configure the "notarize-macos" jobs to only download the artifact they require.
@per1234 per1234 added type: enhancement Proposed improvement topic: code Related to content of the project itself labels Nov 6, 2024
@per1234 per1234 self-assigned this Nov 6, 2024
@per1234 per1234 merged commit 8ae3d80 into arduino:main Nov 6, 2024
16 checks passed
@per1234 per1234 deleted the download-required-artifact branch November 6, 2024 01:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
topic: code Related to content of the project itself type: enhancement Proposed improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant