-
-
Notifications
You must be signed in to change notification settings - Fork 1
improve bugs, performance and consistency #174
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.
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
Conversation
… improved error handling and option for hashtable return
…r): clear field value when empty
…rameter and update error handling
…g for consistency
| } | ||
|
|
||
| function Reset-DatabaseStore{ | ||
| function Reset-DatabaseStore { |
Check warning
Code scanning / PSScriptAnalyzer
Function 'Reset-DatabaseStore' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'. Warning
| } | ||
|
|
||
| function Reset-Database{ | ||
| function Reset-Database { |
Check warning
Code scanning / PSScriptAnalyzer
Function 'Reset-Database' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'. Warning
| [Parameter()][string]$Owner, | ||
| [Parameter()][string]$ProjectNumber, | ||
| [Parameter()][switch]$Force | ||
| [Parameter()][switch]$Force, |
Check warning
Code scanning / PSScriptAnalyzer
The parameter 'Force' has been declared but not used. Warning
|
|
||
| function writeMockCommandFile($Content){ | ||
|
|
||
| $list = $Content | ConvertTo-Json |
Check notice
Code scanning / PSScriptAnalyzer
Line has trailing whitespace Note
| function writeMockCommandFile($Content){ | ||
|
|
||
| $list = $Content | ConvertTo-Json | ||
|
|
Check notice
Code scanning / PSScriptAnalyzer
Line has trailing whitespace Note
| $list = $Content | ConvertTo-Json | ||
|
|
||
| $sorted = $list | Sort-Object fileName | ||
|
|
Check notice
Code scanning / PSScriptAnalyzer
Line has trailing whitespace Note
| } | ||
|
|
||
| function Reset-DatabaseStore{ | ||
| function Reset-DatabaseStore { |
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Reset-DatabaseStore' does not have a help comment. Note
| } Export-ModuleMember -Function Reset-DatabaseStore | ||
|
|
||
| function Get-DatabaseStore{ | ||
| function Get-DatabaseStore { |
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Get-DatabaseStore' does not have a help comment. Note
| } | ||
|
|
||
| function Invoke-ProjectHelperGetDatabaseStorePath{ | ||
| function Invoke-ProjectHelperGetDatabaseStorePath { |
Check notice
Code scanning / PSScriptAnalyzer
The cmdlet 'Invoke-ProjectHelperGetDatabaseStorePath' does not have a help comment. Note
| # TODO: Create a Resolve-ProjectItemByUrl so that we can udpate the Project | ||
| # if item is not cached but exists in projects. | ||
| # TODO: Create a Resolve-ProjectItemByUrl - Depend on function to get item from project remote by url | ||
| # Get-ItemByUrl only check cache so we need a function that will retreive item from project remote |
Check notice
Code scanning / PSScriptAnalyzer
Line has trailing whitespace Note
Enhance database performance by caching the database root and allowing forced resets. Update project item retrieval methods for better error handling and consistency. Improve test cases and error messages for clarity. Refactor code for readability and maintainability.