Skip to content

fix: Remove trailing whitespace in *.ts#2386

Open
notpeter wants to merge 1 commit intoactions:mainfrom
notpeter:trailing_whitespace
Open

fix: Remove trailing whitespace in *.ts#2386
notpeter wants to merge 1 commit intoactions:mainfrom
notpeter:trailing_whitespace

Conversation

@notpeter
Copy link
Copy Markdown

Currently a couple typescript files include trailing whitespace. This makes committing generated js output (necessary for github actions) also have trailing whitespace. Yucky. (example)

Separately, oidc-utils.ts and path-and-artifact-name-validation.ts have trailing whitespace inside template strings used for error strings. This means that both the source has unnecessary trailing whitespace AND the error strings include trailing whitespace and doubled newlines. Double (triple?) yucky.

I think someone misunderstood how newlines / leading whitespace / trailing whitespace in template literals worked.

Before:

Failed to get ID Token. 
 
        Error Code : ${error.statusCode}
 
        Error Message: ${error.message}

After:

Failed to get ID Token.
  Error Code : ${error.statusCode}
  Error Message: ${error.message}

@notpeter notpeter requested review from a team as code owners April 22, 2026 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant