Jenkinsfile improvements #3567
Merged
Jenkinsfile improvements #3567
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
grugnog
force-pushed
the
chown-pr
branch
3 times, most recently
from
496c9c6 to
6113039
Compare
|
This depends on GetDKAN/dkan-tools#298 (and optionally GetDKAN/dkan-tools#297) |
clayliddell
pushed a commit
that referenced
this pull request
Jul 26, 2021
* Always chown on job completion so that subsequent jobs can clean workspace. * Switch to builds for PRs only. * Need actual script. * Check directory exists before getting commit message. * Cleaner fix. * Remove unneeded function. * Test no-proxy branch. * No need to fail if no proxy net connection present.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
fixes 3557
This should resolve the file ownership issues that occurred if DKAN Tools left files owned by root, skip builds for branches (PRs only) and a small amount of other cleanup.