Skip to content

Conversation

@miker83z
Copy link
Contributor

@miker83z miker83z commented Nov 6, 2025

Description of change

Added a generic FlavoredAttribute which can then allow us to define new attributes within the iota_mode compiler part.

Links to any relevant issues

Fixes #9186

How the change has been tested

  • Basic tests (linting, compilation, formatting, unit/integration tests)
  • Patch-specific tests (correctness, functionality coverage)
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that new and existing unit tests pass locally with my changes

@miker83z miker83z self-assigned this Nov 6, 2025
@miker83z miker83z requested a review from a team as a code owner November 6, 2025 17:53
@vercel
Copy link

vercel bot commented Nov 6, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

6 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
apps-backend Ignored Ignored Nov 6, 2025 5:53pm
apps-ui-kit Ignored Ignored Nov 6, 2025 5:53pm
iota-evm-bridge Ignored Ignored Nov 6, 2025 5:53pm
iota-multisig-toolkit Ignored Ignored Nov 6, 2025 5:53pm
rebased-explorer Ignored Ignored Nov 6, 2025 5:53pm
wallet-dashboard Ignored Ignored Nov 6, 2025 5:53pm

@iota-ci iota-ci added sc-platform Issues related to the Smart Contract Platform group. vm-language Issues related to the VM & Language Team labels Nov 6, 2025
// match self {
// Self::Attribute(a) => a.name(),
//}
unimplemented!()
Copy link
Contributor

@Dkwcs Dkwcs Nov 7, 2025

Choose a reason for hiding this comment

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

Why this is unimplemented and the function below as well?

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah I see, you’re modifying it here

@Dkwcs Dkwcs requested a review from valeriyr November 9, 2025 19:45
Copy link
Contributor

@Dkwcs Dkwcs left a comment

Choose a reason for hiding this comment

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

LGTM

@miker83z miker83z merged commit edd0007 into vm-lang/aa-auth/8116-feature-branch Nov 11, 2025
36 of 42 checks passed
@miker83z miker83z deleted the vm-lang/9186-flavored-attributes branch November 11, 2025 11:40
miker83z added a commit that referenced this pull request Nov 27, 2025
…e attr logic into iota_mode (#9187)

# Description of change

Added a generic FlavoredAttribute which can then allow us to define new
attributes within the iota_mode compiler part.

## Links to any relevant issues

Fixes #9186

## How the change has been tested

- [ ] Basic tests (linting, compilation, formatting, unit/integration
tests)
- [ ] Patch-specific tests (correctness, functionality coverage)
- [ ] I have added tests that prove my fix is effective or that my
feature works
- [ ] I have checked that new and existing unit tests pass locally with
my changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sc-platform Issues related to the Smart Contract Platform group. vm-language Issues related to the VM & Language Team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[move] Enable a custom attribute framework based on flavor in the compiler

5 participants