Merge pull request #2064 from codysoyland/ci-updates

Update CirleCI build with Go 1.13 and run enterprise tests

.circleci/config.yml

@@ -2,7 +2,7 @@ version: 2
 defaults: &defaults
   working_directory: /go/src/github.com/pilosa/pilosa
   docker:
-    - image: circleci/golang:1.12
+    - image: circleci/golang:1.13
   environment:
     GO111MODULE: "on"
 fast-checkout: &fast-checkout
@@ -33,7 +33,7 @@ jobs:
     <<: *defaults
     steps:
       - *fast-checkout
-      - run: curl -sfL https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s v1.13.2
+      - run: curl -sfL https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s v1.18.0
       - run: sudo cp bin/golangci-lint /usr/local/bin/
       - run: make golangci-lint
   test-build-arm:
@@ -44,19 +44,19 @@ jobs:
       - run: make build GOOS=linux GOARCH=arm GOARM=6
       - run: make build GOOS=linux GOARCH=arm GOARM=7
       - run: make build GOOS=linux GOARCH=arm64
-  test-golang-1.12: &base-test
+  test-golang-1.13: &base-test
     <<: *defaults
     steps:
       - *fast-checkout
       - run: sudo apt-get install lsof
       - run: make test
-  test-golang-1.12-shard22: &base-test
+  test-golang-1.13-shard22:
     <<: *defaults
     steps:
       - *fast-checkout
       - run: sudo apt-get install lsof
       - run: make test SHARD_WIDTH=22
-  test-golang-1.12-race:
+  test-golang-1.13-race:
     <<: *defaults
     steps:
       - *fast-checkout
@@ -64,11 +64,25 @@ jobs:
       - run:
           command: make test TESTFLAGS="-race -v -timeout=30m"
           no_output_timeout: 30m
-  test-golang-1.12-386:
+  test-golang-1.13-386:
     <<: *base-test
     environment:
       GO111MODULE: "on"
       GOARCH: 386
+  test-golang-1.13-enterprise:
+    <<: *defaults
+    steps:
+      - *fast-checkout
+      - run: sudo apt-get install lsof
+      - run: make test ENTERPRISE=1
+  test-golang-1.12:
+    <<: *defaults
+    docker:
+      - image: circleci/golang:1.12
+    steps:
+      - *fast-checkout
+      - run: sudo apt-get install lsof
+      - run: make test
   test-golang-1.11:
     <<: *defaults
     docker:
@@ -135,15 +149,21 @@ workflows:
       - test-build-arm:
           requires:
             - setup
+      - test-golang-1.13-enterprise:
+          requires:
+            - setup
+      - test-golang-1.13-race:
+          requires:
+            - setup
+      - test-golang-1.13-386:
+          requires:
+            - setup
+      - test-golang-1.13:
+          requires:
+            - setup
       - test-golang-1.12:
           requires:
             - setup
-      - test-golang-1.12-race:
-          requires:
-            - setup
-      - test-golang-1.12-386:
-          requires:
-            - setup
       - test-golang-1.11:
           requires:
             - setup
@@ -154,12 +174,12 @@ workflows:
           requires:
             - linter
             - check-license-headers
-            - test-golang-1.12
+            - test-golang-1.13
       - release:
           requires:
             - linter
             - check-license-headers
-            - test-golang-1.12
+            - test-golang-1.13
           filters:
             tags:
               only: /^v.*/
@@ -172,4 +192,4 @@ workflows:
           requires:
             - linter
             - check-license-headers
-            - test-golang-1.12
+            - test-golang-1.13

go.mod

@@ -43,3 +43,5 @@ require (
 	modernc.org/mathutil v1.0.0
 	modernc.org/strutil v1.0.0
 )
+
+go 1.11

roaring/unmarshal_binary.go

@@ -12,8 +12,6 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-// +build !enterprise
-
 package roaring
 
 import (