Skip to content

frame pallet macro: fix span for error on wrong returned type.#5580

Merged
bkchr merged 5 commits intoparitytech:masterfrom
gui1117:gui-fix-return-type-span
Sep 9, 2024
Merged

frame pallet macro: fix span for error on wrong returned type.#5580
bkchr merged 5 commits intoparitytech:masterfrom
gui1117:gui-fix-return-type-span

Conversation

@gui1117
Copy link
Copy Markdown
Contributor

@gui1117 gui1117 commented Sep 4, 2024

Explicitly give the types in some generated code so that the error shows up good when user code is wrong.

@gui1117 gui1117 requested a review from a team as a code owner September 4, 2024 07:13
@gui1117 gui1117 added the T1-FRAME This PR/Issue is related to core FRAME, the framework. label Sep 9, 2024
Co-authored-by: Bastian Köcher <git@kchr.de>
@bkchr bkchr enabled auto-merge September 9, 2024 20:57
@bkchr bkchr added this pull request to the merge queue Sep 9, 2024
Merged via the queue into paritytech:master with commit aec2b10 Sep 9, 2024
@gui1117 gui1117 deleted the gui-fix-return-type-span branch September 9, 2024 23:58
@gui1117
Copy link
Copy Markdown
Contributor Author

gui1117 commented Sep 9, 2024

@ggwpez do we want to backport this? it happened to me multiple time

github-actions bot pushed a commit that referenced this pull request Sep 11, 2024
Explicitly give the types in some generated code so that the error shows
up good when user code is wrong.

---------

Co-authored-by: Bastian Köcher <git@kchr.de>
(cherry picked from commit aec2b10)
@paritytech-cmd-bot-polkadot-sdk
Copy link
Copy Markdown
Contributor

Successfully created backport PR for stable2407:

github-actions bot pushed a commit that referenced this pull request Sep 16, 2024
Explicitly give the types in some generated code so that the error shows
up good when user code is wrong.

---------

Co-authored-by: Bastian Köcher <git@kchr.de>
(cherry picked from commit aec2b10)
github-actions bot pushed a commit that referenced this pull request Sep 16, 2024
Explicitly give the types in some generated code so that the error shows
up good when user code is wrong.

---------

Co-authored-by: Bastian Köcher <git@kchr.de>
(cherry picked from commit aec2b10)
@paritytech-cmd-bot-polkadot-sdk
Copy link
Copy Markdown
Contributor

Successfully created backport PR for stable2409:

ggwpez pushed a commit that referenced this pull request Sep 16, 2024
Backport #5580 into `stable2407` from gui1117.

See the
[documentation](https://github.com/paritytech/polkadot-sdk/blob/master/docs/BACKPORT.md)
on how to use this bot.

<!--
  # To be used by other automation, do not modify:
  original-pr-number: #${pull_number}
-->

Co-authored-by: Guillaume Thiolliere <gui.thiolliere@gmail.com>
ggwpez pushed a commit that referenced this pull request Sep 17, 2024
Backport #5580 into `stable2409` from gui1117.

See the
[documentation](https://github.com/paritytech/polkadot-sdk/blob/master/docs/BACKPORT.md)
on how to use this bot.

<!--
  # To be used by other automation, do not modify:
  original-pr-number: #${pull_number}
-->

Co-authored-by: Guillaume Thiolliere <gui.thiolliere@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T1-FRAME This PR/Issue is related to core FRAME, the framework.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants