Skip to content

Commit

Permalink
chore: remove release-tests package (#4032)
Browse files Browse the repository at this point in the history
# Description

## Problem\*

Resolves <!-- Link to GitHub Issue -->

## Summary\*

This removes the `release-tests` package as it's relatively low-value
and introduces flakiness into CI.

## Additional Context



## Documentation\*

Check one:
- [x] No documentation needed.
- [ ] Documentation included in this PR.
- [ ] **[Exceptional Case]** Documentation to be submitted in a separate
PR.

# PR Checklist\*

- [x] I have tested the changes locally.
- [x] I have formatted the changes with [Prettier](https://prettier.io/)
and/or `cargo fmt` on default settings.
  • Loading branch information
TomAFrench authored Jan 12, 2024
1 parent 2932017 commit e8e96a7
Show file tree
Hide file tree
Showing 9 changed files with 4 additions and 364 deletions.
20 changes: 0 additions & 20 deletions .github/workflows/publish-nargo.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,16 +73,6 @@ jobs:
path: ./dist/*
retention-days: 3

- name: Install Yarn dependencies
if: matrix.target == 'x86_64-apple-darwin'
uses: ./.github/actions/setup

- name: Test built artifact
if: matrix.target == 'x86_64-apple-darwin'
run: |
cp ./target/${{ matrix.target }}/release/nargo ~/.cargo/bin/
yarn workspace release-tests test
- name: Upload binaries to release tag
uses: svenstaro/upload-release-action@v2
if: ${{ inputs.publish || github.event_name == 'schedule' }}
Expand Down Expand Up @@ -161,16 +151,6 @@ jobs:
path: ./dist/*
retention-days: 3

- name: Install Yarn dependencies
if: startsWith(matrix.target, 'x86_64-unknown-linux')
uses: ./.github/actions/setup

- name: Test built artifact
if: startsWith(matrix.target, 'x86_64-unknown-linux')
run: |
cp ./target/${{ matrix.target }}/release/nargo ~/.cargo/bin/
yarn workspace release-tests test
- name: Upload binaries to release tag
uses: svenstaro/upload-release-action@v2
if: ${{ inputs.publish }}
Expand Down
1 change: 0 additions & 1 deletion Dockerfile.ci
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ COPY ./tooling/noir_js_backend_barretenberg/package.json ./tooling/noir_js_backe
COPY ./tooling/noir_js/package.json ./tooling/noir_js/
COPY ./tooling/noir_codegen/package.json ./tooling/noir_codegen/
COPY ./compiler/integration-tests/package.json ./compiler/integration-tests/
COPY ./release-tests/package.json ./release-tests/
COPY ./docs/package.json ./docs/
RUN yarn --immutable
COPY . .
1 change: 0 additions & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
"tooling/noir_codegen",
"tooling/noir_js_backend_barretenberg",
"acvm-repo/acvm_js",
"release-tests",
"docs"
],
"scripts": {
Expand Down
14 changes: 0 additions & 14 deletions release-tests/package.json

This file was deleted.

49 changes: 0 additions & 49 deletions release-tests/test/6_array.test.js

This file was deleted.

3 changes: 0 additions & 3 deletions release-tests/test/utils/nargo.js

This file was deleted.

11 changes: 0 additions & 11 deletions release-tests/test/utils/zx.js

This file was deleted.

35 changes: 0 additions & 35 deletions release-tests/test/version.test.js

This file was deleted.

Loading

0 comments on commit e8e96a7

Please sign in to comment.