Skip to content

Commit 176e7f5

Browse files
authored
Trim trailing whitespace in all md and yml files (actions#1329)
* Trim non-significant trailing whitespace, add final newlines to md,yml files * Add .editorconfig with basic whitespace conventions
1 parent b6d46c1 commit 176e7f5

20 files changed

+52
-44
lines changed

.editorconfig

+8
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,8 @@
1+
# https://editorconfig.org/
2+
3+
[*]
4+
insert_final_newline = true # ensure all files end with a single newline
5+
trim_trailing_whitespace = true # attempt to remove trailing whitespace on save
6+
7+
[*.md]
8+
trim_trailing_whitespace = false # in markdown, "two trailing spaces" is unfortunately meaningful; it means `<br>`

.gitattributes

+3-3
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
#
2121
# Merging from the command prompt will add diff markers to the files if there
2222
# are conflicts (Merging from VS is not affected by the settings below, in VS
23-
# the diff markers are never inserted). Diff markers may cause the following
23+
# the diff markers are never inserted). Diff markers may cause the following
2424
# file extensions to fail to load in VS. An alternative would be to treat
2525
# these files as binary and thus will always conflict and require user
2626
# intervention with every merge. To do so, just uncomment the entries below
@@ -70,9 +70,9 @@
7070

7171
###############################################################################
7272
# diff behavior for common document formats
73-
#
73+
#
7474
# Convert binary document formats to text before diffing them. This feature
75-
# is only available from the command line. Turn it on by uncommenting the
75+
# is only available from the command line. Turn it on by uncommenting the
7676
# entries below.
7777
###############################################################################
7878
*.doc diff=astextplain

.github/ISSUE_TEMPLATE/enhancement_request.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -24,4 +24,4 @@ If applicable, add a code snippet.
2424
**Additional information**
2525
Add any other context about the feature here.
2626

27-
NOTE: if the feature request has been agreed upon then the assignee will create an ADR. See docs/adrs/README.md
27+
NOTE: if the feature request has been agreed upon then the assignee will create an ADR. See docs/adrs/README.md

.github/workflows/build.yml

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,12 @@ on:
77
- main
88
- releases/*
99
paths-ignore:
10-
- '**.md'
10+
- '**.md'
1111
pull_request:
1212
branches:
1313
- '*'
1414
paths-ignore:
15-
- '**.md'
15+
- '**.md'
1616

1717
jobs:
1818
build:

.github/workflows/codeql.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ jobs:
2828
# languages: go, javascript, csharp, python, cpp, java
2929

3030
- name: Manual build
31-
run : |
31+
run : |
3232
./dev.sh layout Release linux-x64
3333
working-directory: src
3434

.github/workflows/release.yml

+5-5
Original file line numberDiff line numberDiff line change
@@ -5,16 +5,16 @@ on:
55
push:
66
paths:
77
- releaseVersion
8-
8+
99
jobs:
1010
check:
1111
if: startsWith(github.ref, 'refs/heads/releases/') || github.ref == 'refs/heads/main'
1212
runs-on: ubuntu-latest
1313
steps:
1414
- uses: actions/checkout@v2
1515

16-
# Make sure ./releaseVersion match ./src/runnerversion
17-
# Query GitHub release ensure version is not used
16+
# Make sure ./releaseVersion match ./src/runnerversion
17+
# Query GitHub release ensure version is not used
1818
- name: Check version
1919
uses: actions/[email protected]
2020
with:
@@ -42,7 +42,7 @@ jobs:
4242
throw e
4343
}
4444
}
45-
45+
4646
build:
4747
needs: check
4848
outputs:
@@ -152,7 +152,7 @@ jobs:
152152
releaseNote = releaseNote.replace(/<LINUX_ARM64_SHA>/g, '${{needs.build.outputs.linux-arm64-sha}}')
153153
console.log(releaseNote)
154154
core.setOutput('version', runnerVersion);
155-
core.setOutput('note', releaseNote);
155+
core.setOutput('note', releaseNote);
156156
# Create GitHub release
157157
- uses: actions/create-release@master
158158
id: createRelease

docs/automate.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ export RUNNER_CFG_PAT=yourPAT
2121

2222
:point_right: [Sample script here](../scripts/create-latest-svc.sh) :point_left:
2323

24-
Run as a one-liner. NOTE: replace with yourorg/yourrepo (repo level) or just yourorg (org level)
24+
Run as a one-liner. NOTE: replace with yourorg/yourrepo (repo level) or just yourorg (org level)
2525
```bash
2626
curl -s https://raw.githubusercontent.com/actions/runner/main/scripts/create-latest-svc.sh | bash -s yourorg/yourrepo
2727
```
@@ -47,7 +47,7 @@ curl -s https://raw.githubusercontent.com/actions/runner/main/scripts/create-lat
4747

4848
The runner is installed as a service using `systemd` and `systemctl`. Docker does not support `systemd` for service configuration on a container.
4949

50-
## Uninstall running as service
50+
## Uninstall running as service
5151

5252
**Scenario**: Run on a machine or VM ([not container](#why-cant-i-use-a-container)) which automates:
5353

@@ -57,7 +57,7 @@ The runner is installed as a service using `systemd` and `systemctl`. Docker doe
5757

5858
:point_right: [Sample script here](../scripts/remove-svc.sh) :point_left:
5959

60-
Repo level one liner. NOTE: replace with yourorg/yourrepo (repo level) or just yourorg (org level)
60+
Repo level one liner. NOTE: replace with yourorg/yourrepo (repo level) or just yourorg (org level)
6161
```bash
6262
curl -s https://raw.githubusercontent.com/actions/runner/main/scripts/remove-svc.sh | bash -s yourorg/yourrepo
6363
```

docs/checks/actions.md

+5-5
Original file line numberDiff line numberDiff line change
@@ -18,16 +18,16 @@ Make sure the runner has access to actions service for GitHub.com or GitHub Ente
1818

1919
- DNS lookup for api.github.com or myGHES.com using dotnet
2020
- Ping api.github.com or myGHES.com using dotnet
21-
- Make HTTP GET to https://api.github.com or https://myGHES.com/api/v3 using dotnet, check response headers contains `X-GitHub-Request-Id`
21+
- Make HTTP GET to https://api.github.com or https://myGHES.com/api/v3 using dotnet, check response headers contains `X-GitHub-Request-Id`
2222
---
2323
- DNS lookup for vstoken.actions.githubusercontent.com using dotnet
2424
- Ping vstoken.actions.githubusercontent.com using dotnet
25-
- Make HTTP GET to https://vstoken.actions.githubusercontent.com/_apis/health or https://myGHES.com/_services/vstoken/_apis/health using dotnet, check response headers contains `x-vss-e2eid`
25+
- Make HTTP GET to https://vstoken.actions.githubusercontent.com/_apis/health or https://myGHES.com/_services/vstoken/_apis/health using dotnet, check response headers contains `x-vss-e2eid`
2626
---
2727
- DNS lookup for pipelines.actions.githubusercontent.com using dotnet
2828
- Ping pipelines.actions.githubusercontent.com using dotnet
29-
- Make HTTP GET to https://pipelines.actions.githubusercontent.com/_apis/health or https://myGHES.com/_services/pipelines/_apis/health using dotnet, check response headers contains `x-vss-e2eid`
30-
- Make HTTP POST to https://pipelines.actions.githubusercontent.com/_apis/health or https://myGHES.com/_services/pipelines/_apis/health using dotnet, check response headers contains `x-vss-e2eid`
29+
- Make HTTP GET to https://pipelines.actions.githubusercontent.com/_apis/health or https://myGHES.com/_services/pipelines/_apis/health using dotnet, check response headers contains `x-vss-e2eid`
30+
- Make HTTP POST to https://pipelines.actions.githubusercontent.com/_apis/health or https://myGHES.com/_services/pipelines/_apis/health using dotnet, check response headers contains `x-vss-e2eid`
3131

3232
## How to fix the issue?
3333

@@ -42,4 +42,4 @@ Make sure the runner has access to actions service for GitHub.com or GitHub Ente
4242
4343
## Still not working?
4444

45-
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.
45+
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.

docs/checks/git.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -31,4 +31,4 @@ The test also set environment variable `GIT_TRACE=1` and `GIT_CURL_VERBOSE=1` be
3131
3232
## Still not working?
3333

34-
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.
34+
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.

docs/checks/internet.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ Even the runner is configured to GitHub Enterprise Server, the runner can still
1313

1414
- DNS lookup for api.github.com using dotnet
1515
- Ping api.github.com using dotnet
16-
- Make HTTP GET to https://api.github.com using dotnet, check response headers contains `X-GitHub-Request-Id`
16+
- Make HTTP GET to https://api.github.com using dotnet, check response headers contains `X-GitHub-Request-Id`
1717

1818
## How to fix the issue?
1919

@@ -23,4 +23,4 @@ Even the runner is configured to GitHub Enterprise Server, the runner can still
2323
2424
## Still not working?
2525

26-
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.
26+
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.

docs/checks/network.md

+2-2
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ Use a 3rd party tool to make the same requests as the runner did would be a good
2525

2626
- Use `nslookup` to check DNS
2727
- Use `ping` to check Ping
28-
- Use `traceroute`, `tracepath`, or `tracert` to check the network route between the runner and the Actions service
28+
- Use `traceroute`, `tracepath`, or `tracert` to check the network route between the runner and the Actions service
2929
- Use `curl -v` to check the network stack, good for verifying default certificate/proxy settings.
3030
- Use `Invoke-WebRequest` from `pwsh` (`PowerShell Core`) to check the dotnet network stack, good for verifying bugs in the dotnet framework.
3131

@@ -50,7 +50,7 @@ If you are having trouble connecting, try these steps:
5050
- The runner runs on .net core, lets validate the local settings for that stack
5151
- Open up `pwsh`
5252
- Run the command using the urls above `Invoke-WebRequest {url}`
53-
3. If not, get a packet trace using a tool like wireshark and start looking at the TLS handshake.
53+
3. If not, get a packet trace using a tool like wireshark and start looking at the TLS handshake.
5454
- If you see a Client Hello followed by a Server RST:
5555
- You may need to configure your TLS settings to use the correct version
5656
- You should support TLS version 1.2 or later

docs/checks/nodejs.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -27,4 +27,4 @@ All javascript base Actions will get executed by the built-in `node` at `<runner
2727
2828
## Still not working?
2929

30-
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.
30+
Contact GitHub customer service or log an issue at https://github.com/actions/runner if you think it's a runner issue.

docs/checks/sslcert.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ As long as your certificate is generated properly, most of the issues should be
1212
> !!! DO NOT SKIP SSL CERT VALIDATION !!!
1313
> !!! IT IS A BAD SECURITY PRACTICE !!!
1414
15-
### Download SSL certificate chain
15+
### Download SSL certificate chain
1616

1717
Depends on how your SSL server certificate gets configured, you might need to download the whole certificate chain from a machine that has trusted the SSL certificate's CA.
1818

@@ -28,7 +28,7 @@ The actions runner is a dotnet core application which will follow how dotnet loa
2828

2929
You can get full details documentation at [here](https://docs.microsoft.com/en-us/dotnet/standard/security/cross-platform-cryptography#x509store)
3030

31-
In short:
31+
In short:
3232
- Windows: Load from Windows certificate store.
3333
- Linux: Load from OpenSSL CA cert bundle.
3434
- macOS: Load from macOS KeyChain.
@@ -43,13 +43,13 @@ To let the runner trusts your CA certificate, you will need to:
4343
1. RedHat: https://www.redhat.com/sysadmin/ca-certificates-cli
4444
2. Ubuntu: http://manpages.ubuntu.com/manpages/focal/man8/update-ca-certificates.8.html
4545
3. Google search: "trust ca certificate on [linux distribution]"
46-
4. If all approaches failed, set environment variable `SSL_CERT_FILE` to the CA bundle `.pem` file we get.
46+
4. If all approaches failed, set environment variable `SSL_CERT_FILE` to the CA bundle `.pem` file we get.
4747
> To verify cert gets installed properly on Linux, you can try use `curl -v https://sitewithsslissue.com` and `pwsh -Command \"Invoke-WebRequest -Uri https://sitewithsslissue.com\"`
4848
4949
### Trust CA certificate for Git CLI
5050

5151
Git uses various CA bundle file depends on your operation system.
52-
- Git packaged the CA bundle file within the Git installation on Windows
52+
- Git packaged the CA bundle file within the Git installation on Windows
5353
- Git use OpenSSL certificate CA bundle file on Linux and macOS
5454

5555
You can check where Git check CA file by running:

docs/contribute.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ Issues in this repository should be for the runner application. Note that the V
1212

1313
## Enhancements and Feature Requests
1414

15-
We ask that before significant effort is put into code changes, that we have agreement on taking the change before time is invested in code changes.
15+
We ask that before significant effort is put into code changes, that we have agreement on taking the change before time is invested in code changes.
1616

1717
1. Create a feature request. Once agreed we will take the enhancement
1818
2. Create an ADR to agree on the details of the change.
@@ -46,9 +46,9 @@ Tip: Make sure your job can run on this runner. The easiest way is to set `runs-
4646

4747

4848
## Development Life Cycle
49-
If you're using VS Code, you can follow [these](contribute/vscode.md) steps instead.
49+
If you're using VS Code, you can follow [these](contribute/vscode.md) steps instead.
5050

51-
### To Build, Test, Layout
51+
### To Build, Test, Layout
5252

5353
Navigate to the `src` directory and run the following command:
5454

docs/contribute/vscode.md

+3-3
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ These examples use VS Code, but the idea should be similar across all IDEs as lo
44
## Configure
55

66
To successfully start the runner, you need to register it using a repository and a runner registration token.
7-
Run `Configure` first to build the source code and set up the runner in `_layout`.
7+
Run `Configure` first to build the source code and set up the runner in `_layout`.
88
Once it's done creating `_layout`, it asks for the url of your repository and your token in the terminal.
99

1010
Check [Quickstart](../contribute.md#quickstart-run-a-job-from-a-real-repository) if you don't know how to get this token.
@@ -34,7 +34,7 @@ All the configs below can be found in `.vscode/launch.json`.
3434

3535
If you launch `Run` or `Run [build]`, it starts a process called `Runner.Listener`.
3636
This process will receive any job queued on this repository if the job runs on matching labels (e.g `runs-on: self-hosted`).
37-
Once a job is received, a `Runner.Listener` starts a new process of `Runner.Worker`.
37+
Once a job is received, a `Runner.Listener` starts a new process of `Runner.Worker`.
3838
Since this is a diferent process, you can't use the same debugger session debug it.
3939
Instead, a parallel debugging session has to be started, using a different launch config.
4040
Luckily, VS Code supports multiple parallel debugging sessions.
@@ -45,7 +45,7 @@ Because the worker process is usually started by the listener instead of an IDE,
4545
For this reason, `Runner.Worker` can be configured to wait for a debugger to be attached before it begins any actual work.
4646

4747
Set the environment variable `GITHUB_ACTIONS_RUNNER_ATTACH_DEBUGGER` to `true` or `1` to enable this wait.
48-
All worker processes now will wait 20 seconds before they start working on their task.
48+
All worker processes now will wait 20 seconds before they start working on their task.
4949

5050
This gives enough time to attach a debugger by running `Debug Worker`.
5151
If for some reason you have multiple workers running, run the launch config `Attach` instead.

docs/design/auth.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -58,4 +58,4 @@ Authentication in a workflow run to github.com can be accomplished by using the
5858

5959
Hosted runner authentication differs from self-hosted authentication in that runners do not undergo a registration process, but instead, the hosted runners get the OAuth token directly by reading the `.credentials` file. The scope of this particular token is limited for a given workflow job execution, and the token is revoked as soon as the job is finished.
6060

61-
![Hosted runner config and start](../res/hosted-config-start.png)
61+
![Hosted runner config and start](../res/hosted-config-start.png)

docs/start/envlinux.md

+4-4
Original file line numberDiff line numberDiff line change
@@ -27,23 +27,23 @@ Dependencies is missing for Dotnet Core 3.0
2727
Execute ./bin/installdependencies.sh to install any missing Dotnet Core 3.0 dependencies.
2828
```
2929
You can easily correct the problem by executing `./bin/installdependencies.sh`.
30-
The `installdependencies.sh` script should install all required dependencies on all supported Linux versions
30+
The `installdependencies.sh` script should install all required dependencies on all supported Linux versions
3131
> Note: The `installdependencies.sh` script will try to use the default package management mechanism on your Linux flavor (ex. `yum`/`apt-get`/`apt`).
3232
3333
### Full dependencies list
3434

3535
Debian based OS (Debian, Ubuntu, Linux Mint)
3636

3737
- liblttng-ust0
38-
- libkrb5-3
38+
- libkrb5-3
3939
- zlib1g
4040
- libssl1.1, libssl1.0.2 or libssl1.0.0
4141
- libicu63, libicu60, libicu57 or libicu55
4242

4343
Fedora based OS (Fedora, Red Hat Enterprise Linux, CentOS, Oracle Linux 7)
4444

45-
- lttng-ust
46-
- openssl-libs
45+
- lttng-ust
46+
- openssl-libs
4747
- krb5-libs
4848
- zlib
4949
- libicu

docs/start/envosx.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
## Supported Versions
66

77
- macOS High Sierra (10.13) and later versions
8-
8+
99
## Apple Silicon M1
1010

1111
The runner is currently not supported on devices with an Apple M1 chip.

releaseNote.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ mkdir \actions-runner ; cd \actions-runner
2626
# Download the latest runner package
2727
Invoke-WebRequest -Uri https://github.com/actions/runner/releases/download/v<RUNNER_VERSION>/actions-runner-win-x64-<RUNNER_VERSION>.zip -OutFile actions-runner-win-x64-<RUNNER_VERSION>.zip
2828
# Extract the installer
29-
Add-Type -AssemblyName System.IO.Compression.FileSystem ;
29+
Add-Type -AssemblyName System.IO.Compression.FileSystem ;
3030
[System.IO.Compression.ZipFile]::ExtractToDirectory("$PWD\actions-runner-win-x64-<RUNNER_VERSION>.zip", "$PWD")
3131
```
3232

scripts/README.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
# Sample scripts for self-hosted runners
22

33
Here are some examples to work from if you'd like to automate your use of self-hosted runners.
4-
See the docs [here](../docs/automate.md).
4+
See the docs [here](../docs/automate.md).

0 commit comments

Comments
 (0)