-
Notifications
You must be signed in to change notification settings - Fork 30
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
Added unit tests and small code cleanup. #771
Conversation
PR Summary
|
@carstingaxion I also discovered some things we should add to distignore. Updated PR with those. Your zip file helped me discover it :-D |
Preview changes with PlaygroundYou can preview the least recent changes for PR#771 by following one of the links below: |
Maybe this line within the
This should be |
This has been another can of worms, but at least this step number 147 is taken now too. Let’s see how far we can get. I illustrated my last test-PR #775 with some screenshots, to show that the checkout grabs the correct ref before doing the build. |
Description of the Change
Closes #
How to test the Change
Changelog Entry
Credits
Props @username, @username2, ...
Checklist: