Skip to content

Commit

Permalink
Revert "RFC30: Compile time benchmark" (#2756)
Browse files Browse the repository at this point in the history
## Motivation and Context

There seems to be some problem which causes this CI job to fail.
https://github.com/awslabs/smithy-rs/actions/runs/5209247728/jobs/9398874499

Reverting smithy-rs#2617 until we have a clearer picture of why and how
to fix.
  • Loading branch information
hlbarber authored Jun 8, 2023
1 parent a6c3aba commit ccbc9a1
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 111 deletions.
39 changes: 1 addition & 38 deletions .github/workflows/pull-request-bot.yml
Original file line number Diff line number Diff line change
Expand Up @@ -135,45 +135,10 @@ jobs:
run: |
aws s3 cp target/doc "s3://${S3_BUCKET_NAME}/docs/${{ inputs.head_revision }}" --recursive
compiletime-benchmark:
runs-on: ubuntu-latest
name: Run Compiletime Benchmark
permissions:
id-token: write
contents: read
pull-requests: write
outputs:
compiletime-benchmark: ${{ steps.compiletime-benchmark.outputs.compiletime-benchmark }}
steps:
- uses: actions/checkout@v3
- uses: actions/cache@v3
name: Gradle Cache
with:
path: |
~/.gradle/caches
~/.gradle/wrapper
key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle*') }}
restore-keys: |
${{ runner.os }}-gradle-
# JDK is needed to generate code
- name: Set up JDK
uses: actions/setup-java@v3
with:
distribution: corretto
java-package: jdk
java-version: ${{ env.java_version }}
- uses: dtolnay/rust-toolchain@master
with:
toolchain: ${{ env.rust_version }}
- name: run benchmark
id: run-compiletime-benchmark
run: bash tools/ci-scripts/compiletime-benchmark && cat /tmp/compiletime-benchmark.md >> "$GITHUB_OUTPUT"

post-bot-comment:
needs:
- generate-diff
- generate-doc-preview
- compiletime-benchmark
runs-on: ubuntu-latest
name: Post bot comment
permissions:
Expand All @@ -199,8 +164,6 @@ jobs:
issue_number: ${{ inputs.issue_number }},
owner: context.repo.owner,
repo: context.repo.repo,
body: '${{ steps.compiletime-benchmark.outputs.compiletime-benchmark }}\n\n' +
'${{ steps.bot-messages.outputs.codegen-diff }}\n\n' +
body: '${{ steps.bot-messages.outputs.codegen-diff }}\n\n' +
'${{ needs.generate-doc-preview.outputs.bot-message }}\n\n'
})
33 changes: 0 additions & 33 deletions tools/ci-scripts/compiletime-benchmark

This file was deleted.

40 changes: 0 additions & 40 deletions tools/compiletime-benchmark/format.py

This file was deleted.

0 comments on commit ccbc9a1

Please sign in to comment.