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

Allow self in non-unit struct and enum top-level assert #219

Conversation

octylFractal
Copy link
Contributor

Also allows top-level map to use self instead of this.

Complete with documentation!

@csnover csnover self-assigned this Sep 20, 2023
@csnover
Copy link
Collaborator

csnover commented Sep 21, 2023

This was rebased, and is merged now. I added a couple of additional commits:

  1. Tried clarifying the docs and added an example using top-level map
  2. Removed the this-compatibility and sanitised the variable
  3. Fixed auto-derived assertion failure messages from leaking the internal transformation from self to something else, and added a test for that
  4. Fixed nightly lints, which no longer allow message-less panics.

@csnover csnover closed this Sep 21, 2023
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