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

Update CI #124

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Update CI #124

wants to merge 6 commits into from

Conversation

ekohl
Copy link

@ekohl ekohl commented Dec 29, 2023

This updates uses setup-ruby to cache and updates various other smaller things. See individual commits for details.

ekohl and others added 6 commits December 29, 2023 18:03
ruby/setup-ruby has caching built in and is guaranteed to be correct.
The default-branch value is just a placeholder that GitHub doesn't
actually implement.
When either ZABBIX_RUN_ACCEPTANCE or ZABBIX_HOST_URL is set as an
environment variable, it's considered that the user opts in to
acceptance tests.

The acceptance tests previously didn't have the _spec suffix, so they
weren't picked up by the file filter. The acceptance tests are dangerous
on a production system, so they are now flagged with their own type.
Flagging also makes it possible to include specific contexts just for
acceptance tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants