Skip to content

Release 2.9.0#189

Merged
lexnv merged 5 commits into
masterfrom
lexnv/release_2.9
Jul 3, 2023
Merged

Release 2.9.0#189
lexnv merged 5 commits into
masterfrom
lexnv/release_2.9

Conversation

@lexnv

@lexnv lexnv commented Jul 3, 2023

Copy link
Copy Markdown
Contributor

[2.9.0] - 2023-07-03

Changed

  • Expose PortableType as public (#188)

@paritytech/subxt-team

lexnv added 5 commits July 3, 2023 17:55
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
Signed-off-by: Alexandru Vasile <alexandru.vasile@parity.io>
@lexnv lexnv self-assigned this Jul 3, 2023
Comment thread src/portable.rs
let Some(ty) = &param.ty else {
continue
};
let Some(ty) = &param.ty else { continue };

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

hrm, is this the style the rustfmt prefers now? 🤔

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yep, the rustfmt passed on the prior PR and unfortunately after merging it the fmt version changed and now prefers this syntax (from master/CI job).

@lexnv lexnv merged commit cc97f4c into master Jul 3, 2023
@lexnv lexnv deleted the lexnv/release_2.9 branch July 3, 2023 15:27
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.

3 participants