Skip to content
This repository was archived by the owner on Feb 21, 2024. It is now read-only.

Commit 69f3aca

Browse files
authored
Merge pull request #2064 from codysoyland/ci-updates
Update CirleCI build with Go 1.13 and run enterprise tests
2 parents f1216fd + 3db1a91 commit 69f3aca

File tree

3 files changed

+34
-14
lines changed

3 files changed

+34
-14
lines changed

.circleci/config.yml

+32-12
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ version: 2
22
defaults: &defaults
33
working_directory: /go/src/github.com/pilosa/pilosa
44
docker:
5-
- image: circleci/golang:1.12
5+
- image: circleci/golang:1.13
66
environment:
77
GO111MODULE: "on"
88
fast-checkout: &fast-checkout
@@ -33,7 +33,7 @@ jobs:
3333
<<: *defaults
3434
steps:
3535
- *fast-checkout
36-
- run: curl -sfL https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s v1.13.2
36+
- run: curl -sfL https://install.goreleaser.com/github.com/golangci/golangci-lint.sh | sh -s v1.18.0
3737
- run: sudo cp bin/golangci-lint /usr/local/bin/
3838
- run: make golangci-lint
3939
test-build-arm:
@@ -44,31 +44,45 @@ jobs:
4444
- run: make build GOOS=linux GOARCH=arm GOARM=6
4545
- run: make build GOOS=linux GOARCH=arm GOARM=7
4646
- run: make build GOOS=linux GOARCH=arm64
47-
test-golang-1.12: &base-test
47+
test-golang-1.13: &base-test
4848
<<: *defaults
4949
steps:
5050
- *fast-checkout
5151
- run: sudo apt-get install lsof
5252
- run: make test
53-
test-golang-1.12-shard22: &base-test
53+
test-golang-1.13-shard22:
5454
<<: *defaults
5555
steps:
5656
- *fast-checkout
5757
- run: sudo apt-get install lsof
5858
- run: make test SHARD_WIDTH=22
59-
test-golang-1.12-race:
59+
test-golang-1.13-race:
6060
<<: *defaults
6161
steps:
6262
- *fast-checkout
6363
- run: sudo apt-get install lsof
6464
- run:
6565
command: make test TESTFLAGS="-race -v -timeout=30m"
6666
no_output_timeout: 30m
67-
test-golang-1.12-386:
67+
test-golang-1.13-386:
6868
<<: *base-test
6969
environment:
7070
GO111MODULE: "on"
7171
GOARCH: 386
72+
test-golang-1.13-enterprise:
73+
<<: *defaults
74+
steps:
75+
- *fast-checkout
76+
- run: sudo apt-get install lsof
77+
- run: make test ENTERPRISE=1
78+
test-golang-1.12:
79+
<<: *defaults
80+
docker:
81+
- image: circleci/golang:1.12
82+
steps:
83+
- *fast-checkout
84+
- run: sudo apt-get install lsof
85+
- run: make test
7286
test-golang-1.11:
7387
<<: *defaults
7488
docker:
@@ -135,13 +149,19 @@ workflows:
135149
- test-build-arm:
136150
requires:
137151
- setup
138-
- test-golang-1.12:
152+
- test-golang-1.13-enterprise:
153+
requires:
154+
- setup
155+
- test-golang-1.13-race:
156+
requires:
157+
- setup
158+
- test-golang-1.13-386:
139159
requires:
140160
- setup
141-
- test-golang-1.12-race:
161+
- test-golang-1.13:
142162
requires:
143163
- setup
144-
- test-golang-1.12-386:
164+
- test-golang-1.12:
145165
requires:
146166
- setup
147167
- test-golang-1.11:
@@ -154,12 +174,12 @@ workflows:
154174
requires:
155175
- linter
156176
- check-license-headers
157-
- test-golang-1.12
177+
- test-golang-1.13
158178
- release:
159179
requires:
160180
- linter
161181
- check-license-headers
162-
- test-golang-1.12
182+
- test-golang-1.13
163183
filters:
164184
tags:
165185
only: /^v.*/
@@ -172,4 +192,4 @@ workflows:
172192
requires:
173193
- linter
174194
- check-license-headers
175-
- test-golang-1.12
195+
- test-golang-1.13

go.mod

+2
Original file line numberDiff line numberDiff line change
@@ -43,3 +43,5 @@ require (
4343
modernc.org/mathutil v1.0.0
4444
modernc.org/strutil v1.0.0
4545
)
46+
47+
go 1.11

roaring/unmarshal_binary.go

-2
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,6 @@
1212
// See the License for the specific language governing permissions and
1313
// limitations under the License.
1414

15-
// +build !enterprise
16-
1715
package roaring
1816

1917
import (

0 commit comments

Comments
 (0)