Skip to content
This repository has been archived by the owner on Apr 9, 2024. It is now read-only.

chore(acir)!: remove UnknownWitness #123

Merged
merged 1 commit into from
Feb 28, 2023
Merged

Conversation

TomAFrench
Copy link
Member

@TomAFrench TomAFrench commented Feb 26, 2023

Related issue(s)

(If it does not already exist, first create a GitHub issue that describes the problem this Pull Request (PR) solves before creating the PR and link it here.)

Resolves (link to issue)

Description

Summary of changes

I've removed the functionally dead code of UnknownWitness as we don't use it anywhere.

It is in the public interface however so this is a breaking change.

Dependency additions / changes

N/A

Test additions / changes

N/A

Checklist

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt with default settings.
  • I have linked this PR to the issue(s) that it resolves.
  • I have reviewed the changes on GitHub, line by line.
  • I have ensured all changes are covered in the description.

Additional context

(If applicable.)

@kevaundray kevaundray added this pull request to the merge queue Feb 28, 2023
Merged via the queue into master with commit 9f002c7 Feb 28, 2023
@github-actions github-actions bot mentioned this pull request Feb 28, 2023
@TomAFrench TomAFrench deleted the remove-unknown-witness branch February 28, 2023 17:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants