starred/cv4pve-api-powershell
1
0
Fork 0
mirror of https://github.com/Corsinvest/cv4pve-api-powershell.git synced 2026-04-25 12:15:48 +03:00
Code Issues Projects Releases 10 Packages Wiki Activity

[PR #50] [MERGED] Bump version to 9.1.2 and improve code quality #51

New issue
Closed
opened 2026-02-26 17:32:25 +03:00 by kerem · 0 comments
kerem commented 2026-02-26 17:32:25 +03:00
Owner
Copy link

📋 Pull Request Information

Original PR: https://github.com/Corsinvest/cv4pve-api-powershell/pull/50
Author: @franklupo
Created: 2/9/2026
Status: ✅ Merged
Merged: 2/9/2026
Merged by: @franklupo

Base: master ← Head: fix-issue-49

📝 Commits (1)

  • 0a55252 Bump version to 9.1.2 and improve code quality

📊 Changes

2 files changed (+4 additions, -9 deletions)

View changed files

📝 Corsinvest.ProxmoxVE.Api/Corsinvest.ProxmoxVE.Api.psd1 (+2 -2)
📝 Corsinvest.ProxmoxVE.Api/Corsinvest.ProxmoxVE.Api.psm1 (+2 -7)

📄 Description

Summary

  • Update module version from 9.1.1 to 9.1.2
  • Fix null comparison to use Yoda condition ($null -ne instead of -ne $null)
  • Refactor IsNumeric function to use PowerShell type casting idiom

Test plan

  • Verify module version is updated to 9.1.2
  • Test null comparisons work correctly
  • Test IsNumeric function with various inputs

Fixes #49

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Corsinvest/cv4pve-api-powershell/pull/50 **Author:** [@franklupo](https://github.com/franklupo) **Created:** 2/9/2026 **Status:** ✅ Merged **Merged:** 2/9/2026 **Merged by:** [@franklupo](https://github.com/franklupo) **Base:** `master` ← **Head:** `fix-issue-49` --- ### 📝 Commits (1) - [`0a55252`](https://github.com/Corsinvest/cv4pve-api-powershell/commit/0a552523046443db73ef1f930d7bc2d0bbbf1d04) Bump version to 9.1.2 and improve code quality ### 📊 Changes **2 files changed** (+4 additions, -9 deletions) <details> <summary>View changed files</summary> 📝 `Corsinvest.ProxmoxVE.Api/Corsinvest.ProxmoxVE.Api.psd1` (+2 -2) 📝 `Corsinvest.ProxmoxVE.Api/Corsinvest.ProxmoxVE.Api.psm1` (+2 -7) </details> ### 📄 Description ## Summary - Update module version from 9.1.1 to 9.1.2 - Fix null comparison to use Yoda condition (`$null -ne` instead of `-ne $null`) - Refactor `IsNumeric` function to use PowerShell type casting idiom ## Test plan - [ ] Verify module version is updated to 9.1.2 - [ ] Test null comparisons work correctly - [ ] Test IsNumeric function with various inputs Fixes #49 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 17:32:25 +03:00
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
gh-pages
switch-license-gpl-to-mit
v9.1.4
v9.1.3
v9.1.2
v9.1.1
v9.1.0
v8.4.1
v8.4.0
v8.3.2
v8.3.1
v8.3.0
v8.2.3
v8.2.2
v8.2.1
v8.2.0
v8.1.2
v8.1.1
v8.0.4
v8.0.3
v8.0.2
v8.0.1
v8.0.0
v7.4.1
v7.1.3
v7.1.2
v1.0.1
v1.0.0
Labels
Clear labels   
bug

   
pull-request

Mirrored from GitHub Pull Request

No labels
bug
pull-request
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/cv4pve-api-powershell#51
No description provided.