Add workaround for git diff issue in perltidy workflow and use updated perl images for all other tests #153
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
nicomen
changed the title
|
Please squash commits so we can merge. |
Add workaround for git diff issue: actions/checkout#766 Fix indentation that perltidy complains about Use Debian tagged images for the the perl images Use newer checkout version to avoid node deprecation warnings
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.
Summary
Setting the current dir as a safe directory in perltidy workflow to be able to run the git diff command without error:
warning: Not a git repository. Use --no-index to compare two paths outside a working tree ext codeUsing Debian tagged perl docker images as the existing perl ones are failing due to using an older format
Motivation
Apparently a security fix in Git made git commands fail, possibly due to different owner of parent directory.
Current perl images yield the following error when pulled:
[DEPRECATION NOTICE] Docker Image Format v1 and Docker Image manifest version 2, schema 1 support is disabled by default and will be removed in an upcoming release. Suggest the author of docker.io/library/perl:5.14 to upgrade the image to the OCI Format or Docker Image manifest v2, schema 2. More information at https://docs.docker.com/go/deprecated-image-specs/ Error: Docker pull failed with exit code 1References
safe directory issue: actions/checkout#766
perl images issue: briandfoy/cpan-audit#58