Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ci): switch to pulling ACIR from artifacts #3043

Closed
wants to merge 5 commits into from

Conversation

TomAFrench
Copy link
Member

This is a companion PR to noir-lang/noir#3288

This switches us over to pulling the ACIR artifacts from an artifact on a workflow run. Instead of closing the Noir repository we look for an artifact called "acir-artifacts" on a workflow attached to the particular branch on the Noir repository.

@TomAFrench
Copy link
Member Author

Oof, this is unfortunate. We need to set up gh but this needs a github PAT to be passed through to circleci.

@ludamad
Copy link
Collaborator

ludamad commented Oct 25, 2023

Can you use the aztec bot token for this?

@TomAFrench
Copy link
Member Author

Yeah, we don't really need any permissions so aztec-bot would work. I'll take a look at this sometime tomorrow/Fri.

* master: (95 commits)
  feat!: make noir_wasm the default compiler (#3090)
  feat: Add root rollup circuit (#3217)
  refactor: Move sumcheck (#3189)
  refactor: Move flavors (#3188)
  refactor: Move honk/pcs (#3187)
  feat: Extract types to a types crate (#3203)
  fix: better update steps for dockerized sandbox (#3204)
  refactor: Move transcripts (#3176)
  chore: Fix typo in aztec sandbox (#3191)
  feat: Enable pkc in noir (#3194)
  chore: more boiler plate code for merge rollup (#3182)
  fix: remove unused import (#3200)
  feat(docs): New getting started flow  (#2957)
  feat!: adding all the (note, nonce) pairs in `PXE.addNote` and hiding `PXE.getNoteNonces` (#3196)
  chore: refactor e2e to not use @aztec/types (#3175)
  chore: bump noir (#3197)
  feat: add cli command update aztec dependencies (#3128)
  feat!: API inconsistency fix (#3190)
  chore: Add initial skeleton code for root/merge/mase rollups for Noir (#3178)
  feat: migrate cpp private kernel tests to noir (#3165)
  ...
@TomAFrench
Copy link
Member Author

It's not clear to me where these tokens are being injected from plus we're going to need it inside an EC2 instance(?) rather than a script being run directly on circleci. I don't think I know enough about the CI setup here to be able to do this quickly tbh.

@TomAFrench TomAFrench closed this Jan 11, 2024
@ludamad ludamad deleted the tf/pull-acir-from-artifacts branch August 22, 2024 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants