Boxes CI tests no longer work #3705
Assignees
Comments
PhilWindle
pushed a commit
that referenced
this issue
Dec 15, 2023
- Fix syntax for checking if contracts were deployed - Fix separator for `TO_TAINT` var list - disable boxes in CI because of #3705 # Checklist: Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge. - [ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag. - [ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code. - [ ] Every change is related to the PR description. - [ ] I have [linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) this pull request to relevant issues (if any exist).
|
I'm giving this thought. Will propose best course of action soon. |
AztecBot
pushed a commit
to AztecProtocol/build-system
that referenced
this issue
Dec 16, 2023
- Fix syntax for checking if contracts were deployed - Fix separator for `TO_TAINT` var list - disable boxes in CI because of AztecProtocol/aztec-packages#3705 # Checklist: Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge. - [ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag. - [ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code. - [ ] Every change is related to the PR description. - [ ] I have [linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) this pull request to relevant issues (if any exist).
|
Resolving as part of: #3727 |
michaelelliot
pushed a commit
to Swoir/noir_rs
that referenced
this issue
Feb 28, 2024
- Fix syntax for checking if contracts were deployed - Fix separator for `TO_TAINT` var list - disable boxes in CI because of AztecProtocol#3705 # Checklist: Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge. - [ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag. - [ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code. - [ ] Every change is related to the PR description. - [ ] I have [linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue) this pull request to relevant issues (if any exist).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
These were previously run by downloading the yarn-project and running a docker-compose with that image running the tests from yarn-project/boxes/....
Since boxes is no longer included in yarn-project directory this doesn't work anymore. In fact no Docker image is being built that includes the boxes currently.
The text was updated successfully, but these errors were encountered: