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

Allow specifying Arduino Lint ref for manual run of integration test #63

Merged
merged 2 commits into from
Aug 10, 2021
Merged

Allow specifying Arduino Lint ref for manual run of integration test #63

merged 2 commits into from
Aug 10, 2021

Conversation

per1234
Copy link
Contributor

@per1234 per1234 commented Aug 10, 2021

When triggered automatically by a push, PR, or schedule event, the integration test uses the latest release of Arduino Lint. That is the same version used by the production engine, so appropriate for testing development on the engine.

However, the integration of Arduino Lint into the engine should also be tested before a new version is released to be immediately used by the production engine. This will be made easier by allowing the integration test to be run via a manual trigger of the GitHub Actions workflow, specifying any arbitrary Git ref of the Arduino Lint repository to build from.

image


Demonstration runs:

@per1234 per1234 added type: enhancement Proposed improvement topic: infrastructure Related to project infrastructure labels Aug 10, 2021
When triggered automatically by a push, PR, or schedule event, the integration test uses the latest release of Arduino
Lint. That is the same version used by the production engine, so appropriate for testing development on the engine.

However, the integration of Arduino Lint into the engine should also be tested before a new version is released to be
immediately used by the production engine. This will be made easier by allowing the integration test to be run via a
manual trigger of GitHub Actions, specifying any arbitrary Git ref of the Arduino Lint repository to build from.
@per1234 per1234 merged commit 5d3fcfa into arduino:main Aug 10, 2021
@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.

2 participants