Skip to content

Conversation

@Morriar
Copy link
Contributor

@Morriar Morriar commented May 8, 2025

This PR introduces a few changes:

  • Signatures can be marked as without_runtime
  • Signature without_runtime are printed as T::Sig::WithoutRuntime.sig
  • Parse signature with T::Sig::WithoutRuntime.sig
  • Handle RBS <> RBI translate for T::Sig::WithoutRuntime.sig

I also spotted an error in the parser where random calls to sig like Foo.sig would be parsed as actual Sig instances while it shouldn't. Fixed those.

Morriar added 5 commits May 8, 2025 16:00
Signed-off-by: Alexandre Terrasa <[email protected]>
Signed-off-by: Alexandre Terrasa <[email protected]>
Signed-off-by: Alexandre Terrasa <[email protected]>
@Morriar Morriar self-assigned this May 8, 2025
@Morriar Morriar requested a review from a team as a code owner May 8, 2025 20:32
@Morriar Morriar added the feature New feature label May 8, 2025
@Morriar Morriar requested a review from egiurleo May 8, 2025 20:35
Copy link
Contributor

@egiurleo egiurleo left a comment

Choose a reason for hiding this comment

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

🥳

@Morriar Morriar merged commit fcae14d into main May 8, 2025
8 checks passed
@Morriar Morriar deleted the at-without-runtime branch May 8, 2025 20:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants