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

Add tests #5

Merged
merged 2 commits into from
Apr 29, 2022
Merged

Add tests #5

merged 2 commits into from
Apr 29, 2022

Conversation

jpkrohling
Copy link
Collaborator

@jpkrohling jpkrohling commented Mar 9, 2022

Add tests for all current distributions.

Requires #4 and closes #1

Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
@jpkrohling jpkrohling marked this pull request as ready for review March 11, 2022 18:20
Signed-off-by: Juraci Paixão Kröhling <juraci@kroehling.de>
@jpkrohling
Copy link
Collaborator Author

@gouthamve, could you please take a look at this one? I'm accepting suggestions to replace the bash code.

@jpkrohling
Copy link
Collaborator Author

@fstab, if you'd like to review this one as well, it would be much appreciated!

@jpkrohling
Copy link
Collaborator Author

@fstab, should I continue pursuing this?

@jpkrohling
Copy link
Collaborator Author

Merging, as we can change it later if this doesn't fit our needs.

@jpkrohling jpkrohling merged commit b0734e4 into main Apr 29, 2022
@jpkrohling jpkrohling deleted the jpkrohling/issue1 branch May 24, 2023 15:30
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.

Write the first e2e tests
1 participant