-
Notifications
You must be signed in to change notification settings - Fork 5
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 CI #20
Add CI #20
Conversation
MichaelStubbings
commented
Aug 6, 2021
- Add Continuous Integration
remove wget install step
Run an echo command to see the namespace resolver used by the Ubuntu VM
Add further directory list tests
Update link to point to tag release over a compare link
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I notice that this merge request has 150 commits! Would it be possible, please, to squash the redundant content?
Other than that my only comments are a few stylistic bash
comments. Do please use the if ...; then
syntax.
remove redundant touch commands use literal cat commands
Cleanup bash syntax
Add ls test commands back into workflow
Comment directory tests
Update artifact upload targets
Looking much cleaner. 5 character indent in |