From c0318a7cdfbb189cdf019e5a60a7455ae7af6714 Mon Sep 17 00:00:00 2001 From: Anton Harniakou Date: Fri, 24 Nov 2017 00:03:19 +0300 Subject: [PATCH] Ignore lines with a .PHONY prefix Closes #24 --- parser/parser.go | 2 ++ parser/parser_test.go | 7 +++++++ 2 files changed, 9 insertions(+) diff --git a/parser/parser.go b/parser/parser.go index 8007f49..e3d2a92 100644 --- a/parser/parser.go +++ b/parser/parser.go @@ -100,6 +100,8 @@ func (p *Parser) parse() error { switch { case p.i == len(p.lines)-1: return nil + case strings.HasPrefix(p.peek(), ".PHONY"): + p.advance() case len(p.peek()) == 0: p.pushComment() p.advance() diff --git a/parser/parser_test.go b/parser/parser_test.go index c0296cf..8edfacc 100644 --- a/parser/parser_test.go +++ b/parser/parser_test.go @@ -43,6 +43,12 @@ size: @gopher-count /tmp/out.js | sort -nr .PHONY: size +.PHONY: dummy +# Just a comment. +# Just another comment. +dummy: + @ls + ` p := parser.New() @@ -63,6 +69,7 @@ size: // parser.Comment{Target:"api", Value:"Start the API server."} // parser.Comment{Target:"deps", Value:"Display dependency graph."} // parser.Comment{Target:"size", Value:"Display size of dependencies.\n\n- foo\n- bar\n- baz"} + // parser.Comment{Target:"dummy", Value:"Just a comment.\nJust another comment."} } func ExampleParser_Parse_withoutComments() {