Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit537af83

Browse files
authored
Only set safe.directory on Cygwin (which needs it)
This stops setting the current directory as an explicit safedirectory on CI for non-Windows systems, where this is not neededbecause the repository has the ownership Git expects. The step nameis updated accordingly to reflect its now narrower purpose.This also adds shell quoting to $(pwd) in the Cygwin workflow. Inpractice, on CI, the path is very unlikely to contain whitespace,but double-quoting $ expansions on which splitting and globbing areunwanted is more robust and better expresses intent. This also hasthe benefit that users who use the CI workflows as a guide tocommands they run locally, where on Windows they may very well havespaces somewhere in this absolute path, will use a correct command.
1 parent830025b commit537af83

File tree

2 files changed

+5
-6
lines changed

2 files changed

+5
-6
lines changed

‎.github/workflows/cygwin-test.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
SHELLOPTS:igncr
1313
TMP:"/tmp"
1414
TEMP:"/tmp"
15-
15+
1616
steps:
1717
-name:Force LF line endings
1818
run:git config --global core.autocrlf input
@@ -24,8 +24,8 @@ jobs:
2424
packages:python39 python39-pip python39-virtualenv git
2525
-name:Tell git to trust this repo
2626
shell:bash.exe -eo pipefail -o igncr "{0}"
27-
run:|
28-
/usr/bin/git config --global --add safe.directory $(pwd)
27+
run:|
28+
/usr/bin/git config --global --add safe.directory"$(pwd)"
2929
/usr/bin/git config --global protocol.file.allow always
3030
-name:Install dependencies and prepare tests
3131
shell:bash.exe -eo pipefail -o igncr "{0}"

‎.github/workflows/pythonpackage.yml

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -52,9 +52,8 @@ jobs:
5252
set -x
5353
mypy -p git
5454
55-
-name:Tell git to trust this repo
56-
run:|
57-
/usr/bin/git config --global --add safe.directory $(pwd)
55+
-name:Tell git to allow file protocol even for submodules
56+
run:|
5857
/usr/bin/git config --global protocol.file.allow always
5958
6059
-name:Test with pytest

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp