Circle CI: skip uploading build artifacts on forks #1093
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Signed-off-by: Anton Bershanskiy 45960703+bershanskiy@users.noreply.github.com
By submitting this pull request, I confirm the following (please check boxes, eg [X]) Failure to fill the template will close your PR:
Please submit all pull requests against the
development
branch. Failure to do so will delay or deny your requestHow familiar are you with the codebase?: {replace this text with a number from 1 to 10, with 1 being not familiar, and 10 being very familiar}
1
When external contributor (like me) forks FTL and enables Circle CI on it, Circle CI tries to upload artifacts to Pi-Hole's FTP server. Of course, upload fails and fails the whole workflow and skips actual test.
Example run before this change:
https://app.circleci.com/pipelines/github/bershanskiy/FTL/1/workflows/9d0fd821-4a83-4b63-826f-60948822beda/jobs/2
Example run after this change:
https://app.circleci.com/pipelines/github/bershanskiy/FTL/2/workflows/32643484-69c6-4465-88d0-8105f7b8a6c8/jobs/15
This template was created based on the work of
udemy-dl
.