Merge branch 'master' into spelling

patrickdevivo · web-flow · commit 1d32243a2d79 · 2019-12-28T20:09:25.000-05:00
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -1,5 +1,5 @@
 name: tests
-on: [push]
+on: [push, pull_request]
 jobs:
 
   build:
diff --git a/README.md b/README.md
@@ -3,7 +3,7 @@
 [![Go Report Card](https://goreportcard.com/badge/github.com/augmentable-dev/tickgit)](https://goreportcard.com/report/github.com/augmentable-dev/tickgit)
 ![GitHub release (latest SemVer)](https://img.shields.io/github/v/release/augmentable-dev/tickgit)
 [![Coverage](http://gocover.io/_badge/github.com/augmentable-dev/tickgit)](http://gocover.io/github.com/augmentable-dev/tickgit)
-[![TODOs](https://img.shields.io/endpoint?url=https%3A%2F%2Ftickgit.augmentable.dev%2Ftodos-badge%3Frepo%3Dhttps%3A%2F%2Fgithub.com%2Faugmentable-dev%2Ftickgit)](https://todos.augmentable.dev/?repo=https://github.com/augmentable-dev/tickgit)
+[![TODOs](https://img.shields.io/endpoint?url=https%3A%2F%2Ftickgit.augmentable.dev%2Ftodos-badge%3Frepo%3Dhttps%3A%2F%2Fgithub.com%2Faugmentable-dev%2Ftickgit)](https://todos.tickgit.com/browse?repo=https://github.com/augmentable-dev/tickgit)
 
 ## tickgit 🎟️
 
@@ -37,6 +37,8 @@ TODO: useTransition hook instead.
 128 TODOs Found 📝
 ```
 
+Check out [an example](https://todos.tickg.it/?repo=https://github.com/kubernetes/kubernetes) of the TODOs tickgit will surface for the Kubernetes codebase.
+
 #### Coming Soon
 
 - [x] History - get a better sense of how old TODOs are, when they were introduced and by whom
diff --git a/pkg/comments/comments.go b/pkg/comments/comments.go
@@ -69,6 +69,7 @@ var LanguageParseOptions map[Language]*lege.ParseOptions = map[Language]*lege.Pa
 	"Swift":        CStyleCommentOptions,
 	"Common Lisp":  LispStyleCommentOptions,
 	"Emacs Lisp":   LispStyleCommentOptions,
+	"R":            HashStyleCommentOptions,
 }
 
 // Comments is a list of comments
diff --git a/pkg/comments/comments_test.go b/pkg/comments/comments_test.go
@@ -13,7 +13,7 @@ func TestJSFiles(t *testing.T) {
 		t.Fatal(err)
 	}
 
-	if len(comments) != 2 {
+	if len(comments) != 3 {
 		t.Fail()
 	}
 }
diff --git a/pkg/comments/testdata/javascript/node_modules/ignored.js b/pkg/comments/testdata/javascript/node_modules/ignored.js
diff --git a/pkg/comments/testdata/javascript/subdir/index.js b/pkg/comments/testdata/javascript/subdir/index.js
@@ -0,0 +1,2 @@
+// this comment should be found
+

Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,7 @@ var LanguageParseOptions map[Language]lege.ParseOptions = map[Language]lege.Pa`
`69`	`69`	`"Swift": CStyleCommentOptions,`
`70`	`70`	`"Common Lisp": LispStyleCommentOptions,`
`71`	`71`	`"Emacs Lisp": LispStyleCommentOptions,`
	`72`	`+ "R": HashStyleCommentOptions,`
`72`	`73`	`}`
`73`	`74`
`74`	`75`	`// Comments is a list of comments`
Original file line number	Diff line number	Diff line change
`@@ -13,7 +13,7 @@ func TestJSFiles(t *testing.T) {`
`13`	`13`	`t.Fatal(err)`
`14`	`14`	`}`
`15`	`15`
`16`		`- if len(comments) != 2 {`
	`16`	`+ if len(comments) != 3 {`
`17`	`17`	`t.Fail()`
`18`	`18`	`}`
`19`	`19`	`}`
Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,2 @@`
	`1`	`+// this comment should be found`
	`2`	`+`