Skip to content
This repository was archived by the owner on Nov 9, 2025. It is now read-only.

Conversation

@Veykril
Copy link
Member

@Veykril Veykril commented Dec 14, 2020

No description provided.

@matklad
Copy link
Contributor

matklad commented Dec 14, 2020

Could check (using [patch.'crates-io']) that this works with rust-analyzer? MIght make sense to prep a draft PR

@Veykril
Copy link
Member Author

Veykril commented Dec 14, 2020

I'm almost certain this will cause problems in RA, would surprise me if not 😄 I can start working on a PR that pulls this in properly first before this gets merged.

@matklad
Copy link
Contributor

matklad commented Dec 16, 2020

bord d+

@Veykril
Copy link
Member Author

Veykril commented Dec 16, 2020

You typo'd bors.

@matklad
Copy link
Contributor

matklad commented Dec 16, 2020

bors d+

@bors
Copy link
Contributor

bors bot commented Dec 16, 2020

✌️ Veykril can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

@Veykril
Copy link
Member Author

Veykril commented Dec 16, 2020

bors r+

@bors
Copy link
Contributor

bors bot commented Dec 16, 2020

Build succeeded:

@bors bors bot merged commit 3ff83b0 into rust-analyzer:master Dec 16, 2020
bors bot added a commit to rust-lang/rust-analyzer that referenced this pull request Dec 16, 2020
6896: Node-ify lifetimes r=jonas-schievink a=Veykril

Let's see if this passes the tests 🤞 

Depends on rust-analyzer/ungrammar#15

Co-authored-by: Jonas Schievink <jonasschievink@gmail.com>
Co-authored-by: Jonas Schievink <jonas.schievink@ferrous-systems.com>
Co-authored-by: Lukas Wirth <lukastw97@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants