Skip to content

Commit 84dd855

Browse files
authored
Merge pull request #182 from rulasg/remove-spaces
style: remove trailing whitespace and improve whitespace handling
2 parents 95735fd + 32a9b96 commit 84dd855

88 files changed

Lines changed: 294 additions & 334 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.devcontainer/devcontainer.json

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
"name": "PowerShell",
66
// Or use a Dockerfile or Docker Compose file. More info: https://containers.dev/guide/dockerfile
77
"image": "mcr.microsoft.com/powershell:lts-ubuntu-22.04",
8-
8+
99
"features": {
1010
"ghcr.io/devcontainers/features/common-utils:2": {},
1111
"ghcr.io/devcontainers/features/sshd:1": {},
@@ -20,10 +20,10 @@
2020
// Configure properties specific to VS Code.
2121
"vscode": {
2222
// Set *default* container specific settings.json values on container create.
23-
"settings": {
23+
"settings": {
2424
"terminal.integrated.defaultProfile.linux": "pwsh"
2525
},
26-
26+
2727
// Add the IDs of extensions you want installed when the container is created.
2828
"extensions": [
2929
"ms-vscode.powershell",
@@ -40,4 +40,3 @@
4040
// Uncomment to connect as root instead. More info: https://aka.ms/dev-containers-non-root.
4141
// "remoteUser": "root"
4242
}
43-

.github/copilot-instructions.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ Disable-ProjectHelperDebug
6060

6161
### GraphQL Integration
6262
1. Store GraphQL in template files: `/public/graphql/queryName.query` or `.mutant`
63-
2. Retrieve via: `Get-GraphQLString "queryName.query"`
63+
2. Retrieve via: `Get-GraphQLString "queryName.query"`
6464
3. Execute: `Invoke-GraphQL -Query $query -Variables $variables`
6565

6666
Example:
@@ -110,7 +110,7 @@ The folder structure in `Test/` must exactly match the structure in the main mod
110110
- `Test_FindProject_SUCCESS` (success case)
111111
- `Test_AddProjectUser_SUCCESS_SingleUser` (specific variant)
112112
- `Test_GetProjectIssue_NotFound` (error case)
113-
- **Conventions**:
113+
- **Conventions**:
114114
- Use PascalCase matching the actual function name (e.g., `Get-SomeInfo``Test_GetSomeInfo_<tip>`)
115115
- `<tip>` should be a descriptive word indicating the test goal (SUCCESS, NotFound, InvalidInput, etc.)
116116
- Use assertions: `Assert-IsTrue`, `Assert-Contains`, `Assert-AreEqual`, `Assert-Count`, `Assert-IsNull`

.github/copilot-pull-request-description-instructions.md

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Pull Request Code Instructions
22

3-
## PR TITLE
3+
## PR TITLE
44

55
Follow this guidelines to construct the title of your pull request.
66

@@ -39,6 +39,3 @@ References:
3939

4040
- Add a summery of the intention of the PR. Use the title and the messages of the commits to create a summary.
4141
- Add a list with all the commit messages in the PR.
42-
43-
44-

.github/workflows/deploy_module_on_release.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -21,17 +21,17 @@ jobs:
2121
2222
- name: Run test.ps1
2323
shell: pwsh
24-
run: |
24+
run: |
2525
$result = ./test.ps1 -ShowTestErrors
26-
26+
2727
Write-Output $result
2828
2929
import-module ./tools/Test_Helper/
3030
3131
$passed = Test-Result -Result $result
3232
3333
if($passed)
34-
{ "All test passed" | Write-Verbose -verbose ; exit 0 }
34+
{ "All test passed" | Write-Verbose -verbose ; exit 0 }
3535
else
3636
{ "Not all tests passed" | Write-Verbose -verbose ; exit 1 }
3737
@@ -42,15 +42,15 @@ jobs:
4242
EVENT_REF: ${{ github.event.ref }}
4343
RELEASE_TAG: ${{ github.event.release.tag_name }}
4444
RELEASE_NAME: ${{ github.event.release.name }}
45-
run: |
45+
run: |
4646
4747
# Import required modules for deployment
4848
import-module ./tools/Test_Helper/
4949
5050
Get-RequiredModule -Verbose | Import-RequiredModule -AllowPrerelease
5151
5252
# GET TAG NAME
53-
53+
5454
## Ref definition. Branch or Tag
5555
$env:EVENT_REF = $env:REF
5656
If ([string]::IsNullOrEmpty($env:EVENT_REF)) {
@@ -70,4 +70,3 @@ jobs:
7070
7171
# DEPLOYMENT
7272
./deploy.ps1 -VersionTag $tag -NugetApiKey $env:NUGETAPIKEY
73-

.github/workflows/powershell.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ name: PSScriptAnalyzer
1212
on:
1313
workflow_dispatch:
1414
pull_request:
15-
15+
1616
permissions:
1717
contents: read
1818

@@ -21,7 +21,7 @@ jobs:
2121
permissions:
2222
contents: read # for actions/checkout to fetch code
2323
security-events: write # for github/codeql-action/upload-sarif to upload SARIF results
24-
actions: read # only required for a private repository by github/codeql-action/upload-sarif to get the Action run status
24+
actions: read # only required for a private repository by github/codeql-action/upload-sarif to get the Action run status
2525
name: PSScriptAnalyzer
2626
runs-on: ubuntu-latest
2727
steps:
@@ -33,15 +33,14 @@ jobs:
3333
# Check https://github.com/microsoft/action-psscriptanalyzer for more info about the options.
3434
# The below set up runs PSScriptAnalyzer to your entire repository and runs some basic security rules.
3535
path: .\
36-
recurse: true
36+
recurse: true
3737
severity: 'Error'
38-
# Include your own basic security rules. Removing this option will run all the rules
38+
# Include your own basic security rules. Removing this option will run all the rules
3939
# includeRule: '"PSAvoidGlobalAliases", "PSAvoidUsingConvertToSecureStringWithPlainText"'
4040
output: results.sarif
41-
41+
4242
# Upload the SARIF file generated in the previous step
4343
- name: Upload SARIF results file
4444
uses: github/codeql-action/upload-sarif@v3
4545
with:
4646
sarif_file: results.sarif
47-

.github/workflows/test_with_TestingHelper.yml

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -31,17 +31,16 @@ jobs:
3131
# Runs a single command using the runners shell
3232
- name: Run test.ps1
3333
shell: pwsh
34-
run: |
34+
run: |
3535
$result = ./test.ps1 -ShowTestErrors
36-
36+
3737
Write-Output $result
3838
3939
import-module ./tools/Test_Helper/
4040
4141
$passed = Test-Result -Result $result
4242
4343
if($passed)
44-
{ "All test passed" | Write-Verbose -verbose ; exit 0 }
45-
else
44+
{ "All test passed" | Write-Verbose -verbose ; exit 0 }
45+
else
4646
{ "Not all tests passed" | Write-Verbose -verbose ; exit 1 }
47-

.vscode/launch.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
// For more information, visit: https://go.microsoft.com/fwlink/?linkid=830387
55
//
66
// Module V2 - https://raw.githubusercontent.com/rulasg/DemoPsModule/main/.vscode/launch.json
7-
7+
88
"version": "0.2.0",
99
"configurations": [
1010
{

Notes.md

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ Notes used during the development of the module
77
```mermaid
88
graph TD;
99
10-
10+
1111
1212
subgraph Module
1313
0([Sync-ProjectItemStaged])
@@ -33,7 +33,7 @@ graph TD;
3333
H <--> I
3434
3535
36-
36+
3737
```
3838

3939
## Update-ProjectDatabase
@@ -50,10 +50,10 @@ graph TD;
5050
5151
end
5252
I<-->G
53-
53+
5454
5555
U<-- GitHubOrgProjectWithFields--> I[InvokeHelper]
5656
G <--> H[Invoke-RestMethod]
5757
H <--> Q([GitHub GraphQL API])
58-
59-
```
58+
59+
```

ProjectHelper.psd1

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,4 +129,3 @@ PrivateData = @{
129129
# DefaultCommandPrefix = ''
130130

131131
}
132-

ProjectHelper.psm1

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,4 +10,3 @@ $MODULE_PATH = $PSScriptRoot
1010
catch { Write-Error -Message "Failed to import $($import.fullname): $_" }
1111
}
1212
}
13-

0 commit comments

Comments
 (0)