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

Alloy 0.8.3 #163

Closed
wants to merge 4 commits into from
Closed

Alloy 0.8.3 #163

wants to merge 4 commits into from

Conversation

rory-ocl
Copy link
Contributor

Description

This upgrades the alloy version to 0.8.3.

This alloy version includes generic integer types, so instead of implementing those on our end, we special case the built-in integer sizes so they work with both primitive ints and alloy generics.

This PR also includes some unit tests for encoding and decoding to ensure that the alloy upgrade does not break this functionality.

Checklist

  • I have documented these changes where necessary.
  • I have read the DCO and ensured that these changes comply.
  • I assign this work under its open source licensing.

rorytrent and others added 4 commits September 16, 2024 17:06
These tests are important before upgrading alloy to avoid misuing their
API and causing errors.
This alloy version includes generic integer types, so instead of
implementing those on our end, we special case the built-in integer
sizes so they work with both primitive ints and alloy generics.
chrisco512
chrisco512 previously approved these changes Oct 9, 2024
@rory-ocl rory-ocl dismissed chrisco512’s stale review October 10, 2024 17:00

The merge-base changed after approval.

@joshuacolvin0
Copy link
Member

already merged in other PR

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.

4 participants