search

PlagueHO / CosmosDB

PowerShell Module for working with Azure Cosmos DB databases, collections, documents, attachments, offers, users, permissions, triggers, stored procedures and user defined functions.
http://dscottraynsford.com
MIT License
152 stars 46 forks source link

Drop Support for Windows PowerShell 5.1 #443

Open PlagueHO opened 2 years ago

PlagueHO commented 2 years ago

Question: Should support be dropped for PowerShell 5.1 and below?

This will simplify some code and eliminate the need to test on PS 5.1 as part of the CI/CD processes. Some changes proposed would prevent the module from working on PowerShell 5.1 so would need to be announced that it is no longer supported.

See #442

Szeraax commented 2 years ago

Implementing MaxRetires and RetryDelay like what Pwsh does would happen BEFORE the 429 error handling. There is no benefit to a PR for #442 in removing support for Windows Powershell. A PR for 442 will do a regular loop at lower priority than the 429 handler.