-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Proposal: Add Dockerfile.spot? #6716
Conversation
Codecov Report
@@ Coverage Diff @@
## dev #6716 +/- ##
===================================
Coverage 0% 0%
===================================
Files 11 11
Lines 145 145
Branches 11 11
===================================
Misses 145 145 Continue to review full report at Codecov.
|
Mind add a few comments in the source? |
Sure. |
Yes. So that whoever read the code in the future understand what is this for. |
Merge once CI passes (CI actually does nothing regarding this change. Just for the formality) |
Forgot you still have the merge privilege. |
Ah my bad, thought your comment was directed at me! |
Old habit die hard... |
Support creating 'Spots' directly from PRs in the repo.