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

fix: Remove automatic update to AztecProtocol/dev-bb.js #1712

Merged
merged 1 commit into from
Aug 21, 2023

Conversation

kevaundray
Copy link
Contributor

Related to #1711

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 this pull request to relevant issues (if any exist).

@kevaundray kevaundray enabled auto-merge (squash) August 21, 2023 20:42
@AztecBot AztecBot disabled auto-merge August 21, 2023 20:46
@AztecBot
Copy link
Collaborator

@kevaundray kevaundray merged commit 6969f6d into master Aug 21, 2023
@kevaundray kevaundray deleted the kw/remove-dev-bb-js branch August 21, 2023 20:56
dan-aztec added a commit that referenced this pull request Aug 22, 2023
fix colorscheme

fix banner

aztec logo

prettier working?

add api endpoint

switch to cjs file

parse contract functions, but no working form

delete unneeded text

comments

slightly cleaner form

slightly better css

fix: Remove automatic update to `AztecProtocol/dev-bb.js` (#1712)

Related to #1711

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).

git subrepo push --branch=master circuits/cpp/barretenberg

subrepo:
  subdir:   "circuits/cpp/barretenberg"
  merged:   "d883900f9"
upstream:
  origin:   "https://github.com/AztecProtocol/barretenberg"
  branch:   "master"
  commit:   "d883900f9"
git-subrepo:
  version:  "0.4.6"
  origin:   "???"
  commit:   "???"

chore: sync bb master (#1713)

Ran:

```
./scripts/git_subrepo.sh pull circuits/cpp/barretenberg
git checkout origin/master -- .gitmodules
```

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).

copy the contract ABI into the new subdirectory

fix output json name
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.

3 participants