erikdarlingdata / DarlingData

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

Refactor stored procedure for improved readability and debugging. #456

Closed Imran-imtiaz48 closed 1 week ago

Imran-imtiaz48 commented 1 week ago

"Refactored stored procedure 'dbo.DebugExample' to enhance readability and facilitate debugging:

Why: The refactor was done to improve the maintainability and clarity of the stored procedure code. By reorganizing and simplifying the code structure and comments, it becomes easier for developers to understand, debug, and modify the procedure as needed in the future. These changes aim to enhance productivity and reduce potential errors during development and maintenance phases.

erikdarlingdata commented 1 week ago

@Imran-imtiaz48 please read the contributing guide -- you've pointed this PR against the incorrect branch, and you didn't open a corresponding issue to discuss the changes first.

But, just to be honest with you, I disagree with nearly every single change you've made here. If you find your version better, you are of course free to keep and maintain that fork. I do not find these changes helpful, though.

Thanks!