Skip to content

Conversation

@matklad
Copy link
Contributor

@matklad matklad commented May 4, 2021

A lot of APIs we use in this release!

@lnicola
Copy link
Member

lnicola commented May 6, 2021

bors try

@bors
Copy link
Contributor

bors bot commented May 6, 2021

try

Merge conflict.

@SomeoneToIgnore
Copy link
Contributor

I've resolved the handlers.rs conflict by accepting my changes from https://github.com/rust-analyzer/rust-analyzer/pull/8711/files#diff-13acc206c18467a7dbe2876d204fee460c728232fbb677a4d769641e76dd9fe5 and discarding the 1.52 ones, since the parsing function had been changed entirely.

Yet I've done it via the GitHub UI and that apparently introduced the merge commit instead of the rebase: if that's unwanted, we need a proper rebase.

@Veykril
Copy link
Member

Veykril commented May 6, 2021

Merge commits will be rejected by CI, so ye we want a proper rebase here instead

@SomeoneToIgnore
Copy link
Contributor

Well, I'll take it over since started to mess with it, hope it's fine.

@SomeoneToIgnore
Copy link
Contributor

bors try

bors bot added a commit that referenced this pull request May 6, 2021
@matklad
Copy link
Contributor Author

matklad commented May 6, 2021

Well, I'll take it over since started to mess with it

Thanks!

@bors
Copy link
Contributor

bors bot commented May 6, 2021

@SomeoneToIgnore
Copy link
Contributor

bors r+

@bors
Copy link
Contributor

bors bot commented May 6, 2021

@bors bors bot merged commit b37b709 into rust-lang:master May 6, 2021
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.

4 participants