erikdarlingdata / DarlingData

Open source SQL Server nonsense: sp_PressureDetector, sp_QuickieStore, sp_HumanEvents, etc.
https://www.erikdarling.com/
MIT License
483 stars 137 forks source link

sp_QuickieStore: Several of the parameters don't appear in the example file. #489

Open ReeceGoding opened 2 days ago

ReeceGoding commented 2 days ago

Is your feature request related to a problem? Please describe. A pet peeve of mine is having parameters that aren't in the documentation's list of examples. Currently, the following are in the @help = 1 text but not the examples:

Furthermore, the following are in @help = 1 but not the readme:

Describe the solution you'd like Write up some examples and typical use cases, then add them in.

Describe alternatives you've considered Self-pity. A few of these are my fault!

Are you ready to build the code for the feature? Probably, but I don't know if I can be bothered to spin up a 2022 box just for the sake of getting familiar with the parameters exclusive to that.

IMPORTANT: If you're going to contribute code, please read the contributing guide first. https://github.com/erikdarlingdata/DarlingData/blob/main/CONTRIBUTING.md Yep.

erikdarlingdata commented 1 day ago

@ReeceGoding yes, keeping up with the documentation is an admin task that I don't do a great job with. It seems to only happen in avalanches. Anything you want to do in here is appreciated.

erikdarlingdata commented 8 hours ago

@ReeceGoding should this be closed as well?

ReeceGoding commented 7 hours ago

@erikdarlingdata No. The recent merge made a tiny dent in it while documenting the regression stuff, but it didn't touch the vast majority of the stuff here.