Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Ensure macro-internal variables are properly encapsulated #109

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

dead-claudia
Copy link
Contributor

I actually ran into this in the wild with a local Metadata struct that wasn't being referenced correctly in the outer function, resulting in pretty obvious compilation errors.

@epage epage merged commit edc6d92 into rust-cli:master Jun 13, 2023
10 of 12 checks passed
@epage
Copy link
Collaborator

epage commented Jun 13, 2023

Thanks!

@epage
Copy link
Collaborator

epage commented Jun 13, 2023

Released in v1.1.5

dead-claudia added a commit to dead-claudia/human-panic that referenced this pull request Jun 13, 2023
The other tests only needed modified to fix a column number.

This should help prevent regressions down this front. I've verified
they also failed prior to that patch.
dead-claudia added a commit to dead-claudia/human-panic that referenced this pull request Jun 13, 2023
The other tests only needed modified to fix a column number.

This should help prevent regressions down this front. I've verified
they also failed prior to that patch.
dead-claudia added a commit to dead-claudia/human-panic that referenced this pull request Jun 13, 2023
The other tests only needed modified to fix a column number. The
action was mysteriously only running one of the tests - I've fixed
that.

This should help prevent regressions for rust-cli#109 and possibly others.
I've verified they also failed prior to that patch.
dead-claudia added a commit to dead-claudia/human-panic that referenced this pull request Jun 13, 2023
The other tests only needed modified to fix a column number. The
action was mysteriously only running one of the tests - I've fixed
that.

This should help prevent regressions for rust-cli#109 and possibly others.
I've verified they also failed prior to that patch.
dead-claudia added a commit to dead-claudia/human-panic that referenced this pull request Jun 13, 2023
The other tests only needed modified to fix a column number. The
action was mysteriously only running one of the tests - I've fixed
that.

This should help prevent regressions for rust-cli#109 and possibly others.
I've verified they also failed prior to that patch.
dead-claudia added a commit to dead-claudia/human-panic that referenced this pull request Jun 13, 2023
The other tests only needed modified to fix a column number. The
action was mysteriously only running one of the tests - I've fixed
that.

This should help prevent regressions for rust-cli#109 and possibly others.
I've verified they also failed prior to that patch.
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.

None yet

2 participants