Skip to content

Conversation

@m4rch3n1ng
Copy link
Contributor

similar to #12759, except for auto-pairs instead of comment tokens. unblocks #14696.

@m4rch3n1ng m4rch3n1ng force-pushed the injection-auto-pairs branch from c806e75 to 6294bcd Compare October 31, 2025 19:50
@m4rch3n1ng m4rch3n1ng force-pushed the injection-auto-pairs branch from 6294bcd to 6091dc4 Compare November 3, 2025 14:37
@m4rch3n1ng m4rch3n1ng force-pushed the injection-auto-pairs branch 2 times, most recently from f4b58b6 to ba49f86 Compare November 3, 2025 14:58
editor: &'a Editor,
loader: &'a syntax::Loader,
view: &View,
doc: &Document,
Copy link
Member

Choose a reason for hiding this comment

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

since this is on Document we can use self instead

Copy link
Contributor Author

Choose a reason for hiding this comment

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

whoops

@m4rch3n1ng m4rch3n1ng force-pushed the injection-auto-pairs branch from ba49f86 to 1f3f1b8 Compare November 3, 2025 15:06
@m4rch3n1ng
Copy link
Contributor Author

oh the failing ci is due to updates on the master branch, i was so confused

@m4rch3n1ng
Copy link
Contributor Author

also what did i change to make that fail now? i didn't even touch the typed.rs file??

why is this failing? what did i do? i am so confused.
@m4rch3n1ng m4rch3n1ng force-pushed the injection-auto-pairs branch from 1f3f1b8 to 2a74127 Compare November 3, 2025 15:12
@the-mikedavis
Copy link
Member

The typed module uses use super::* so the use arc_swap::access::DynAccess; added here must've changed the call to load there. Odd that the CI caught that without needing to rebase though

@m4rch3n1ng
Copy link
Contributor Author

Odd that the CI caught that without needing to rebase though

yeah did not know, that it did that, but i had the same happen to me over in a different pr like an hour ago.

@the-mikedavis the-mikedavis merged commit 46af400 into helix-editor:master Nov 3, 2025
7 checks passed
@m4rch3n1ng m4rch3n1ng deleted the injection-auto-pairs branch November 3, 2025 15:23
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.

2 participants