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(ssa): Rename ObjectType::Pointer to ObjectType::ArrayPointer #1077

Merged
merged 2 commits into from
Mar 31, 2023

Conversation

joss-aztec
Copy link
Contributor

Related issue(s)

Resolves #757

Description

Summary of changes

Renames ObjectType::Pointer to more explicit ObjectType::ArrayPointer

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.

Documentation needs

  • This PR requires documentation updates when merged.

Additional context

kevaundray
kevaundray previously approved these changes Mar 31, 2023
@kevaundray kevaundray added this pull request to the merge queue Mar 31, 2023
Merged via the queue into master with commit aee8368 Mar 31, 2023
@kevaundray kevaundray deleted the joss/rename-array-point branch March 31, 2023 16:02
TomAFrench added a commit that referenced this pull request Apr 3, 2023
* master:
  chore: handle public parameters and return values separately in evaluator (#1062)
  chore(ssa): Rename ObjectType::Pointer to ObjectType::ArrayPointer (#1077)
  chore(ssa): indent NumericType into ObjectType (#810)
  chore: apply spelling fixes (#1073)
  feat: Implement `std::unsafe::zeroed` (#1048)
  fix: crash when typechecking fields that don't exist (#1070)
  feat: Implement arrays of structs (#1068)
  feat(nargo): split `nargo` into core and cli packages (#1065)
  chore: split wasm crate into submodules (#1066)
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.

Consider renaming node::ObjectType::Pointer to node::ObjectType::ArrayPointer
2 participants