Skip to content

Sync "Test Go" CI workflow with template #62

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 8 commits into from
Aug 12, 2021
Merged

Sync "Test Go" CI workflow with template #62

merged 8 commits into from
Aug 12, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 12, 2021

We have assembled a collection of reusable GitHub Actions workflows:
https://github.com/arduino/tooling-project-assets
These workflows will be used in the repositories of all Arduino tooling projects.

Some minor improvements and standardizations have been made in the upstream "template" workflow, and those are introduced to this repository via this pull request.

This makes it easier to understand the structure of this long, complex command.
This is the template workflow filename, which is intended to serve as a unique identifier within a collection that might
include multiple related variants, and thus must be a bit more verbose.
There are two file extensions in common use for YAML files: `.yaml` and `.yml`. Although this project uses `.yml`
exclusively for YAML files, this is a standardized workflow which might be applied to projects that have established the
use of the other extension. It will be most flexible if it supports both.
The unit tests don't have any test data at the moment, but it might at some point in the future and this is a
standardized "template" workflow that is intended to be generally applicable to any project.
No functional change, and neither is necessarily superior, but this is how it is in the "template", and so it must be
here as well.
This is the naming convention established in the standardized template workflow. The tool name is "Task", with
"taskfile" being its configuration file, so the previous step name was inaccurate.
This will make it easier for the maintainers to sync fixes and improvements in either direction between the upstream
"template" workflow and its installation in this repository.
@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Aug 12, 2021
@per1234 per1234 merged commit e0774cf into arduino:main Aug 12, 2021
@per1234 per1234 deleted the test-go branch August 12, 2021 04:42
@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.

1 participant