-
Notifications
You must be signed in to change notification settings - Fork 38
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
* disable CircleCI * bump go.mod to Go 1.16 and run go fix * bump go.mod to Go 1.16 and run go fix * run go mod tidy * run gofmt -s * add .github/workflows/automerge.yml * add .github/workflows/go-test.yml * add .github/workflows/go-check.yml * style(testplans): fix staticcheck error * test(responsemanager): fix flaky TestCancellationQueryInProgress resolve possible failure from request finishing early -- simply to fix for TestCancellationViaCommand * test(responsemanager): fix flaky TestValidationAndExtensions Co-authored-by: web3-bot <web3-bot@users.noreply.github.com> Co-authored-by: hannahhoward <hannah@hannahhoward.net>
- Loading branch information
3 people
authored
Oct 8, 2021
1 parent
1c9de96
commit c5420cf
Showing
11 changed files
with
214 additions
and
82 deletions.
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,53 @@ | ||
| # File managed by web3-bot. DO NOT EDIT. | ||
| # See https://github.com/protocol/.github/ for details. | ||
|
|
||
| # Automatically merge pull requests opened by web3-bot, as soon as (and only if) all tests pass. | ||
| # This reduces the friction associated with updating with our workflows. | ||
|
|
||
| on: [ pull_request ] | ||
| name: Automerge | ||
|
|
||
| jobs: | ||
| automerge-check: | ||
| if: github.event.pull_request.user.login == 'web3-bot' | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| status: ${{ steps.should-automerge.outputs.status }} | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| with: | ||
| fetch-depth: 0 | ||
| - name: Check if we should automerge | ||
| id: should-automerge | ||
| run: | | ||
| for commit in $(git rev-list --first-parent origin/${{ github.event.pull_request.base.ref }}..${{ github.event.pull_request.head.sha }}); do | ||
| committer=$(git show --format=$'%ce' -s $commit) | ||
| echo "Committer: $committer" | ||
| if [[ "$committer" != "web3-bot@users.noreply.github.com" ]]; then | ||
| echo "Commit $commit wasn't committed by web3-bot, but by $committer." | ||
| echo "::set-output name=status::false" | ||
| exit | ||
| fi | ||
| done | ||
| echo "::set-output name=status::true" | ||
| automerge: | ||
| needs: automerge-check | ||
| runs-on: ubuntu-latest | ||
| # The check for the user is redundant here, as this job depends on the automerge-check job, | ||
| # but it prevents this job from spinning up, just to be skipped shortly after. | ||
| if: github.event.pull_request.user.login == 'web3-bot' && needs.automerge-check.outputs.status == 'true' | ||
| steps: | ||
| - name: Wait on tests | ||
| uses: lewagon/wait-on-check-action@bafe56a6863672c681c3cf671f5e10b20abf2eaa # v0.2 | ||
| with: | ||
| ref: ${{ github.event.pull_request.head.sha }} | ||
| repo-token: ${{ secrets.GITHUB_TOKEN }} | ||
| wait-interval: 10 | ||
| running-workflow-name: 'automerge' # the name of this job | ||
| - name: Merge PR | ||
| uses: pascalgn/automerge-action@741c311a47881be9625932b0a0de1b0937aab1ae # v0.13.1 | ||
| env: | ||
| GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}" | ||
| MERGE_LABELS: "" | ||
| MERGE_METHOD: "squash" | ||
| MERGE_DELETE_BRANCH: true |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,74 @@ | ||
| # File managed by web3-bot. DO NOT EDIT. | ||
| # See https://github.com/protocol/.github/ for details. | ||
|
|
||
| on: [push, pull_request] | ||
| name: Go Checks | ||
|
|
||
| jobs: | ||
| unit: | ||
| runs-on: ubuntu-latest | ||
| name: All | ||
| env: | ||
| RUNGOGENERATE: false | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| with: | ||
| submodules: recursive | ||
| - uses: actions/setup-go@v2 | ||
| with: | ||
| go-version: "1.17.x" | ||
| - name: Run repo-specific setup | ||
| uses: ./.github/actions/go-check-setup | ||
| if: hashFiles('./.github/actions/go-check-setup') != '' | ||
| - name: Read config | ||
| if: hashFiles('./.github/workflows/go-check-config.json') != '' | ||
| run: | | ||
| if jq -re .gogenerate ./.github/workflows/go-check-config.json; then | ||
| echo "RUNGOGENERATE=true" >> $GITHUB_ENV | ||
| fi | ||
| - name: Install staticcheck | ||
| run: go install honnef.co/go/tools/cmd/staticcheck@df71e5d0e0ed317ebf43e6e59cf919430fa4b8f2 # 2021.1.1 (v0.2.1) | ||
| - name: Check that go.mod is tidy | ||
| uses: protocol/multiple-go-modules@v1.2 | ||
| with: | ||
| run: | | ||
| go mod tidy | ||
| if [[ -n $(git ls-files --other --exclude-standard --directory -- go.sum) ]]; then | ||
| echo "go.sum was added by go mod tidy" | ||
| exit 1 | ||
| fi | ||
| git diff --exit-code -- go.sum go.mod | ||
| - name: gofmt | ||
| if: ${{ success() || failure() }} # run this step even if the previous one failed | ||
| run: | | ||
| out=$(gofmt -s -l .) | ||
| if [[ -n "$out" ]]; then | ||
| echo $out | awk '{print "::error file=" $0 ",line=0,col=0::File is not gofmt-ed."}' | ||
| exit 1 | ||
| fi | ||
| - name: go vet | ||
| if: ${{ success() || failure() }} # run this step even if the previous one failed | ||
| uses: protocol/multiple-go-modules@v1.2 | ||
| with: | ||
| run: go vet ./... | ||
| - name: staticcheck | ||
| if: ${{ success() || failure() }} # run this step even if the previous one failed | ||
| uses: protocol/multiple-go-modules@v1.2 | ||
| with: | ||
| run: | | ||
| set -o pipefail | ||
| staticcheck ./... | sed -e 's@\(.*\)\.go@./\1.go@g' | ||
| - name: go generate | ||
| uses: protocol/multiple-go-modules@v1.2 | ||
| if: (success() || failure()) && env.RUNGOGENERATE == 'true' | ||
| with: | ||
| run: | | ||
| git clean -fd # make sure there aren't untracked files / directories | ||
| go generate ./... | ||
| # check if go generate modified or added any files | ||
| if ! $(git add . && git diff-index HEAD --exit-code --quiet); then | ||
| echo "go generated caused changes to the repository:" | ||
| git status --short | ||
| exit 1 | ||
| fi | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,55 @@ | ||
| # File managed by web3-bot. DO NOT EDIT. | ||
| # See https://github.com/protocol/.github/ for details. | ||
|
|
||
| on: [push, pull_request] | ||
| name: Go Test | ||
|
|
||
| jobs: | ||
| unit: | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| os: [ "ubuntu", "windows", "macos" ] | ||
| go: [ "1.16.x", "1.17.x" ] | ||
| env: | ||
| COVERAGES: "" | ||
| runs-on: ${{ matrix.os }}-latest | ||
| name: ${{ matrix.os}} (go ${{ matrix.go }}) | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| with: | ||
| submodules: recursive | ||
| - uses: actions/setup-go@v2 | ||
| with: | ||
| go-version: ${{ matrix.go }} | ||
| - name: Go information | ||
| run: | | ||
| go version | ||
| go env | ||
| - name: Run repo-specific setup | ||
| uses: ./.github/actions/go-test-setup | ||
| if: hashFiles('./.github/actions/go-test-setup') != '' | ||
| - name: Run tests | ||
| uses: protocol/multiple-go-modules@v1.2 | ||
| with: | ||
| run: go test -v -coverprofile module-coverage.txt ./... | ||
| - name: Run tests (32 bit) | ||
| if: ${{ matrix.os != 'macos' }} # can't run 32 bit tests on OSX. | ||
| uses: protocol/multiple-go-modules@v1.2 | ||
| env: | ||
| GOARCH: 386 | ||
| with: | ||
| run: go test -v ./... | ||
| - name: Run tests with race detector | ||
| if: ${{ matrix.os == 'ubuntu' }} # speed things up. Windows and OSX VMs are slow | ||
| uses: protocol/multiple-go-modules@v1.2 | ||
| with: | ||
| run: go test -v -race ./... | ||
| - name: Collect coverage files | ||
| shell: bash | ||
| run: echo "COVERAGES=$(find . -type f -name 'module-coverage.txt' | tr -s '\n' ',' | sed 's/,$//')" >> $GITHUB_ENV | ||
| - name: Upload coverage to Codecov | ||
| uses: codecov/codecov-action@51d810878be5422784e86451c0e7c14e5860ec47 # v2.0.2 | ||
| with: | ||
| files: '${{ env.COVERAGES }}' | ||
| env_vars: OS=${{ matrix.os }}, GO=${{ matrix.go }} |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,6 +1,6 @@ | ||
| module github.com/ipfs/go-graphsync | ||
|
|
||
| go 1.13 | ||
| go 1.16 | ||
|
|
||
| require ( | ||
| github.com/gogo/protobuf v1.3.2 | ||
|
|
||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.