Skip to content

Update to literal-escaper 0.0.4 #20012

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lnicola
Copy link
Member

@lnicola lnicola commented Jun 16, 2025

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jun 16, 2025
@lnicola
Copy link
Member Author

lnicola commented Jun 16, 2025

r? @hkBst

@rustbot
Copy link
Collaborator

rustbot commented Jun 16, 2025

Failed to set assignee to hkBst: invalid assignee

Note: Only org members with at least the repository "read" role, users with write permissions, or people who have commented on the PR may be assigned.

@lnicola lnicola force-pushed the bump-literal-escaper branch from 2d4675b to 13a46ea Compare June 16, 2025 12:12
@hkBst
Copy link
Member

hkBst commented Jun 16, 2025

Thanks, @lnicola, maybe next time I'll try it myself.

There is one semantic change, that you may want to be aware of, which is that instead of only preserving the last unescape error, now all unescape errors are collected in a Vec before being copied to the enclosing error Vec.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants