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!: Remove backend field from artifacts #3819

Merged
merged 15 commits into from
Dec 16, 2023
Merged

Conversation

kevaundray
Copy link
Contributor

@kevaundray kevaundray commented Dec 15, 2023

Description

In kw/remove-partial-backend-code we removed the opcodes supported feature, which means that artifacts are no longer compiled based on a backends feature set.

Closes #1388

Problem*

Resolves

Summary*

Additional Context

Documentation*

Check one:

  • No documentation needed.
  • Documentation included in this PR.
  • [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

  • I have tested the changes locally.
  • I have formatted the changes with Prettier and/or cargo fmt on default settings.

kevaundray and others added 10 commits December 14, 2023 12:54
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
Co-authored-by: Tom French <15848336+TomAFrench@users.noreply.github.com>
@kevaundray kevaundray marked this pull request as ready for review December 15, 2023 13:26
Copy link
Member

@TomAFrench TomAFrench left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Base automatically changed from kw/remove-partial-backend-code to master December 15, 2023 17:39
@kevaundray kevaundray added this pull request to the merge queue Dec 16, 2023
Merged via the queue into master with commit fa1cf5f Dec 16, 2023
30 checks passed
@kevaundray kevaundray deleted the kw/remove-backend-field branch December 16, 2023 21:09
kevaundray added a commit that referenced this pull request Dec 18, 2023
🤖 I have created a release *beep* *boop*
---


<details><summary>0.22.0</summary>

## [0.22.0](v0.21.0...v0.22.0)
(2023-12-18)


### ⚠ BREAKING CHANGES

* Remove unused methods on ACIR opcodes
([#3841](#3841))
* Remove backend field from artifacts
([#3819](#3819))
* Remove partial backend feature
([#3805](#3805))

### Features

* Add context-centric based API for noir_wasm
([#3798](#3798))
([19155d0](19155d0))


### Miscellaneous Chores

* Remove backend field from artifacts
([#3819](#3819))
([fa1cf5f](fa1cf5f))
* Remove partial backend feature
([#3805](#3805))
([0383100](0383100))
* Remove unused methods on ACIR opcodes
([#3841](#3841))
([9e5d0e8](9e5d0e8))
</details>

<details><summary>0.38.0</summary>

## [0.38.0](v0.37.1...v0.38.0)
(2023-12-18)


### ⚠ BREAKING CHANGES

* Remove unused methods on ACIR opcodes
([#3841](#3841))
* Remove partial backend feature
([#3805](#3805))

### Features

* Aztec-packages
([#3754](#3754))
([c043265](c043265))
* Speed up transformation of debug messages
([#3815](#3815))
([2a8af1e](2a8af1e))


### Bug Fixes

* Deserialize odd length hex literals
([#3747](#3747))
([4000fb2](4000fb2))


### Miscellaneous Chores

* Remove partial backend feature
([#3805](#3805))
([0383100](0383100))
* Remove unused methods on ACIR opcodes
([#3841](#3841))
([9e5d0e8](9e5d0e8))
</details>

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Retrieve BACKEND_IDENTIFIER from backend (instead of hardcoding)
2 participants