Removing lingering GopherJS support debt (#267)

* rename files to more clearly denote just JS requirement * cleanup build tags (debt from legacy GopherJS support) - fixes an issue reported by rsc / Russ Cox in #261 * denote Go 1.14+ requirement with type-checking (possible now due to removing GopherJS support) - fixes #266
hexops · Aug 16, 2020 · a2df9d3 · a2df9d3
1 parent 3dfb52c
commit a2df9d3
Show file tree

Hide file tree

Showing 6 changed files with 17 additions and 17 deletions.
diff --git a/dom_wasmjs_gopherjs.go → dom_js.go b/dom_wasmjs_gopherjs.go → dom_js.go
@@ -1,4 +1,4 @@
-// +build go1.12,wasm,js js
+// +build js
 
 package vecty
 

diff --git a/dom_native.go b/dom_native.go
@@ -1,4 +1,4 @@
-// +build go1.12,!wasm,!js
+// +build !js
 
 package vecty
 

diff --git a/require_go_1_12.go b/require_go_1_12.go
diff --git a/require_go_1_14.go b/require_go_1_14.go
@@ -0,0 +1,13 @@
+// +build !go1.14
+
+package vecty
+
+// Typechecking will pass except one error:
+//
+// 	# github.com/gopherjs/vecty
+// 	../../require_go_1_14.go:7:2: undefined: VECTY_REQUIRES_GO_1_14_PLUS
+//
+
+func init() {
+	VECTY_REQUIRES_GO_1_14_PLUS = "Vecty requires Go 1.14+"
+}
diff --git a/testsuite_wasmjs_test.go → testsuite_js_test.go b/testsuite_wasmjs_test.go → testsuite_js_test.go
@@ -1,4 +1,4 @@
-// +build go1.12,wasm,js js
+// +build js
 
 package vecty
 

diff --git a/testsuite_native_test.go b/testsuite_native_test.go
@@ -1,4 +1,4 @@
-// +build go1.12,!wasm,!js
+// +build !js
 
 package vecty