Skip to content

RPM build check #648

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 10 commits into from
Sep 10, 2022
Merged

RPM build check #648

merged 10 commits into from
Sep 10, 2022

Conversation

hecko
Copy link

@hecko hecko commented Sep 7, 2022

Workflow that builds, installs and test-runs bats on AlmaLinux 8

Migrates #646 from Ubuntu runner to AlmaLinux container
Also contains #645

@martin-schulze-vireso
Copy link
Member

I made the install use the local version, because it would download the latest release otherwise. The test failures are due to install.sh not being installed (which is expected) and due to something with alma not working in the timeout test. This can be reproduced without using RPM, so I think it is unrelated.

@hecko
Copy link
Author

hecko commented Sep 9, 2022

Is there anything else I can do here to support this merge?

@martin-schulze-vireso
Copy link
Member

martin-schulze-vireso commented Sep 9, 2022

I only need to find time to finish this. I am thinking about using the new tagging feature to exclude the setup tests.

Maybe you can look into the reason for the Alma related error. It is locally reproducible in a container.

@martin-schulze-vireso martin-schulze-vireso merged commit 0e8d70b into bats-core:master Sep 10, 2022
@martin-schulze-vireso
Copy link
Member

Thanks for your contribution.

@hecko
Copy link
Author

hecko commented Sep 11, 2022

Thanks for your contribution.

Thanks for maintaining bats so well.

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