pretty 0.1 → 0.2

data/bin/pretty

@@ -19,11 +19,12 @@ else
 		  	
 		#check for shebang
 		if file.readline[0..1] != "#!" 
-			errors[err_cnt] << "#{filename}:#{ln_cnt}: No shebang"
+			errors[err_cnt] = "#{filename}:#{ln_cnt}: No shebang found but it's not end of world ;-)"
 		  	err_cnt+=1
 		end
 		  	
-		#now checking rest of file  	
+		#now checking for rest errors
+		file.rewind	
 		file.each_line do |line|
 			ln_cnt+=1
 		  	if line.size > 80 #short lines are better to read 
@@ -38,6 +39,11 @@ else
 		  		errors[err_cnt] = "#{filename}:#{ln_cnt}: Too many tabulations"
 		  		err_cnt+=1
 		  	end
+		  	if line =~ /[$]\w+/ #it's not good to use global variables		
+		  	errors[err_cnt] = "#{filename}:#{ln_cnt}: It's not good to use global variables.\
+		  	\nIgnore it if you are using $_ or another special variable"
+		  		err_cnt+=1
+		  	end  	
 		end
 		  	
 		file.close
@@ -48,6 +54,8 @@ else
 	#shows and save report
 	name = "report#{Time.now.to_i}.txt"
 	file = File.open(name,"w")
+	p "Found #{err_cnt} errors in #{ARGV.size} files"
+	if err_cnt == 0 : p "This is propably the best program in the world!" end
 	errors.each {|err| p err; file << (err+"\n")}
 	p "Report file: #{name}"
 	file.close

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: pretty
 version: !ruby/object:Gem::Version 
-  version: "0.1"
+  version: "0.2"
 platform: ruby
 authors: 
 - Mateusz 'WujcioL' Tybura
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2008-08-09 00:00:00 +02:00
+date: 2008-08-12 00:00:00 +02:00
 default_executable: 
 dependencies: []