Skip to content

Fix/pssa shouldprocess #36

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

Merged
jfaurskov merged 9 commits into from
Apr 10, 2026
Merged

Fix/pssa shouldprocess #36

Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
b163f14
fix: resolve PSScriptAnalyzer warnings and cost report null issue
lvlindv Apr 9, 2026
af1ce3f
fix: use SuppressMessageAttribute instead of SupportsShouldProcess fo…
lvlindv Apr 9, 2026
d9f4db1
fix: add OutputType attribute to New-Worksheet
lvlindv Apr 9, 2026
8dbf098
Update 7-Report/Get-Report.ps1
lvlindv Apr 10, 2026
2f9de47
Update 7-Report/Get-Report.ps1
lvlindv Apr 10, 2026
da3f0a6
Remove CmdletBinding from Update-SKUPropertySet
lvlindv Apr 10, 2026
f2f982e
Refactor Set-ColumnColor function parameters
lvlindv Apr 10, 2026
86d7556
Refactor New-Worksheet function attributes
lvlindv Apr 10, 2026
241a4f0
Refactor Set-SvcAvailReportObj function
lvlindv Apr 10, 2026
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
4 changes: 2 additions & 2 deletions 1-Collect/Get-AzureServices.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -216,7 +216,7 @@ Function Invoke-CostReportSchedule {
[Parameter(Mandatory = $true)] [string]$SubscriptionId
)
$uri = "https://management.azure.com/subscriptions/$($SubscriptionId)/providers/Microsoft.CostManagement/generateCostDetailsReport?api-version=2025-03-01"
$startDate = (Get-Date).AddDays(-1)
$startDate = (Get-Date).AddDays(-30)
$endDate = (Get-Date)
# Define the request body
$body = @{
Expand Down Expand Up @@ -356,7 +356,7 @@ $baseResult | ForEach-Object {
ipAddress = $ipAddress
meterIds = @()
}
If ($includeCost) {
If ($includeCost -and $null -ne $costDetails) {
Get-MeterId -ResourceId $resourceId -csvObject $costDetails
# add meterIds to the output object
$outObject.meterIds += $meterIds
Expand Down
Loading