Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Retain Rust code items from mod decorated with subxt attribute #721

Merged
merged 7 commits into from
Nov 23, 2022

Conversation

Xanewok
Copy link
Contributor

@Xanewok Xanewok commented Nov 21, 2022

Closes #346

@@ -50,4 +50,5 @@ fn ui_tests() {
fn ui_fail() {
let t = trybuild::TestCases::new();
t.compile_fail("src/incorrect/*.rs");
t.pass("src/correct/*.rs");
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nit: Should we move this to the ui_tests category? Having it under ui_fail might be misleading

Copy link
Member

Choose a reason for hiding this comment

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

or just move this one to ui_tests and keep ui_fall

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, didn't spot the passing cases above; will move it

Copy link
Collaborator

@lexnv lexnv left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Contributor

@ascjones ascjones left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

👍

Xanewok and others added 2 commits November 23, 2022 11:09
Co-authored-by: Andrew Jones <ascjones@gmail.com>
@Xanewok Xanewok merged commit 8b46f5c into master Nov 23, 2022
@Xanewok Xanewok deleted the igor-retain-mod-items branch November 23, 2022 11:28
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Retain Rust code items from mod decorated with subxt attribute
4 participants