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.

Sign up for GitHub

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

Jump to bottom

Add basic acceptance tests #17

Merged
merged 7 commits into from
Mar 2, 2022
Merged

Add basic acceptance tests #17

merged 7 commits into from
Mar 2, 2022

Conversation

smortex
Copy link
Collaborator

@smortex smortex commented Feb 17, 2022 •
edited
Loading

@smortex smortex changed the title acceptance Add basic acceptance tests Feb 17, 2022
@smortex smortex force-pushed the acceptance branch 7 times, most recently from 7242fad to 9fe5d5b Compare February 17, 2022 01:13
This was referenced Feb 17, 2022
Merged
Merged
faxm0dem
faxm0dem approved these changes Feb 28, 2022
View reviewed changes
Copy link
Member

@faxm0dem faxm0dem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.
Also Consider #23 but that might be in another PR

@smortex
Fix CI
af11980
@smortex
Only set major version for Debian / RedHat
196687f 
Acceptance tests do not cope with major.minor for these operating
systems.
@smortex
Add basic acceptance tests
a343a0d
@smortex smortex mentioned this pull request Mar 1, 2022
Merged
@faxm0dem faxm0dem merged commit a343a0d into master Mar 2, 2022
@smortex smortex deleted the acceptance branch March 2, 2022 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@faxm0dem faxm0dem faxm0dem approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@smortex @faxm0dem