Skip to content
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.

Sign up for GitHub

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

Escape -- with double quotes (powershell compatibility) #88

Merged
merged 2 commits into from
Jun 6, 2023
Merged

Escape -- with double quotes (powershell compatibility) #88

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
73364f1
Escape -- with double quotes (powershell compatibility)
ikappaki Jun 2, 2023
a29c590
Disable doctor eldev-file-owners test, add notes
ikappaki Jun 6, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 13 additions & 1 deletion .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,19 @@ jobs:
# Run `doctor' on the project itself. Git hooks are not going
# to be installed in this checkout. Also, don't insist on
# recent stable releases here.
./bin/eldev -p -dtTC doctor -- -githooks -recent-stable-releases
#
# Notes
#
# 1. The `eldev-file-owners` test is disabled on MS-Windows
# because the test expects the onwer of the cache to match the
# user name, but it is instead set to the administrators group
# on the GH CI, see discussion in
# https://github.com/doublep/eldev/pull/88.
#
# 2. The `--` is escaped with `"` for cross platform
# compatibility because in PowerShell `--` is a special
# operator.
./bin/eldev -S "(when (eq system-type 'windows-nt) (push 'eldev-file-owners eldev-doctor-disabled-tests))" -p -dtTC doctor "--" -githooks -recent-stable-releases
env:
ELDEV_LOCAL: "."

Expand Down