search

DotNetAnalyzers / TSqlAnalyzer

T-SQL Code Analyzer for Visual Studio 2015 CTP6
MIT License
57 stars 12 forks source link

updated to also analyze on commandText property #4

Closed RonnieHegelund closed 9 years ago

ErikEJ commented 9 years ago

@RonnieHegelund Would be nice with an accompanying unit test to demonstrate the added feature!

RonnieHegelund commented 9 years ago

I know :-) I think we should have unit tests for every feature, so this will be next step ;-)

-----Original Message----- From: "Erik Ejlskov Jensen" notifications@github.com Sent: ‎11/‎17/‎2014 6:58 AM To: "ErikEJ/TSqlAnalyzer" TSqlAnalyzer@noreply.github.com Cc: "Ronnie Hegelund" ronnie.h.hegelund@gmail.com Subject: Re: [TSqlAnalyzer] updated to also analyze on commandText property(#4)

@RonnirHegelund Would be nice with an accompanying unit test to demonstrate the added feature! — Reply to this email directly or view it on GitHub.=

ErikEJ commented 9 years ago

We have some already for current features...

Sendt fra min Windows Phone

From: Ronnie Hegelundmailto:notifications@github.com Sent: ‎17/‎11/‎2014 07:18 To: ErikEJ/TSqlAnalyzermailto:TSqlAnalyzer@noreply.github.com Cc: Erik Ejlskov Jensenmailto:ejlskov@hotmail.com Subject: Re: [TSqlAnalyzer] updated to also analyze on commandText property (#4)

I know :-) I think we should have unit tests for every feature, so this will be next step ;-)

-----Original Message----- From: "Erik Ejlskov Jensen" notifications@github.com Sent: ‎11/‎17/‎2014 6:58 AM To: "ErikEJ/TSqlAnalyzer" TSqlAnalyzer@noreply.github.com Cc: "Ronnie Hegelund" ronnie.h.hegelund@gmail.com Subject: Re: [TSqlAnalyzer] updated to also analyze on commandText property(#4)

@RonnirHegelund Would be nice with an accompanying unit test to demonstrate the added feature! — Reply to this email directly or view it on GitHub.=

Reply to this email directly or view it on GitHub: https://github.com/ErikEJ/TSqlAnalyzer/pull/4#issuecomment-63264205