search

angrycuban13 / Just-A-Bunch-Of-Starr-Scripts

Just a bunch of scripts for Starr apps
92 stars 10 forks source link

Rework `unattended` logic #50

Closed angrycuban13 closed 1 year ago

angrycuban13 commented 1 year ago

Previous unattended logic could potentially run into an infinite loop where all the same movies would continue to get selected to have their tag removed and re-added and the library would never get fully searched. This will instead remove the tag from all movies which have it if filteredMovies equals 0 and kick off search for n number of items.

bakerboy448 commented 1 year ago

Conflicted :O

angrycuban13 commented 1 year ago

Conflicted :O

Over quote marks of everything lol

github-actions[bot] commented 1 year ago

PSScriptAnalyzer results:

Errors: [0], Warnings: [2], Information: [0]

``` RuleName : PSUseShouldProcessForStateChangingFunctions Severity : Warning ScriptName : upgradinatorr.ps1 Line : 251 Message : Function 'Remove-Tag' has verb that could change system state. Therefore, the function has to support 'ShouldProcess'. RuleName : PSUseSingularNouns Severity : Warning ScriptName : upgradinatorr.ps1 Line : 147 Message : The cmdlet 'Get-ArrItems' uses a plural noun. A singular noun should be used instead. ```