Skip to content

[agent] chore(PSScriptAnalyzer): add PSScriptAnalyzer on CI#145

Merged
RomuDeuxfois merged 6 commits intorelease/currentfrom
improvment/PSScriptAnalyzer
Mar 20, 2026
Merged

[agent] chore(PSScriptAnalyzer): add PSScriptAnalyzer on CI#145
RomuDeuxfois merged 6 commits intorelease/currentfrom
improvment/PSScriptAnalyzer

Conversation

@RomuDeuxfois
Copy link
Member

No description provided.

@RomuDeuxfois RomuDeuxfois force-pushed the improvment/PSScriptAnalyzer branch from bae6c80 to 1a7fced Compare March 6, 2026 16:22
@EmilieFo17 EmilieFo17 requested a review from damgouj March 18, 2026 08:52
Copy link
Member

@damgouj damgouj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a quick remark but tested OK.
Thanks for the powershell linter, we had problems sometimes according to the Windows version/arch/... and have now the right Powershell commands are great !

- run:
name: Lint PowerShell installer scripts (PSScriptAnalyzer)
shell: powershell.exe
command: .\installer\windows\Run-Lint.ps1
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

suggestion: could you clarify in the README how to launch it in local dev env ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot can you add the demands from the comment on the code ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@damgouj we added it :)
Thank you for the review !

Copy link

Copilot AI commented Mar 20, 2026

@RomuDeuxfois I've opened a new pull request, #148, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot AI and others added 3 commits March 20, 2026 16:24
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: RomuDeuxfois <53513584+RomuDeuxfois@users.noreply.github.com>
@RomuDeuxfois RomuDeuxfois merged commit 737be86 into release/current Mar 20, 2026
8 of 9 checks passed
@RomuDeuxfois RomuDeuxfois deleted the improvment/PSScriptAnalyzer branch March 20, 2026 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants