Skip to content

Commit d85e885

Browse files
Publish docker containers (#6)
* Remove select statement * Persist client/build directory
1 parent 88a1214 commit d85e885

2 files changed

Lines changed: 7 additions & 3 deletions

File tree

.circleci/config.yml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,10 @@ jobs:
9191
paths:
9292
- ./client/node_modules
9393
key: v1-dependencies-{{ checksum "./client/package.json" }}
94+
- persist_to_workspace:
95+
root: /go/src/github.com/palantir/bulldozer
96+
paths:
97+
- client/build
9498
- *store-test-results
9599

96100
changelog:
@@ -111,6 +115,8 @@ jobs:
111115
- *install-docker-client
112116
- run: ./godelw publish bintray --url https://api.bintray.com --subject palantir --repository releases --user "$BINTRAY_USERNAME" --password "$BINTRAY_PASSWORD" --publish --downloads-list bulldozer
113117
- run: sha256sum dist/*.tgz
118+
- attach_workspace:
119+
at: /go/src/github.com/palantir/bulldozer
114120
- *build-docker-image
115121
- run: ./publish-docker.sh
116122

github/github.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -398,9 +398,7 @@ func (client *Client) PullRequestForSHA(repo *github.Repository, SHA string) (*g
398398

399399
ticker := time.NewTicker(5 * time.Second)
400400
for {
401-
select {
402-
case <-ticker.C:
403-
}
401+
<-ticker.C
404402

405403
// polling for merge status (https://developer.github.com/v3/pulls/#get-a-single-pull-request)
406404
p, _, err := client.PullRequests.Get(client.Ctx, owner, name, pullRequest.GetNumber())

0 commit comments

Comments
 (0)