Skip to content

Drop old extension method syntax #9476

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

Merged
merged 5 commits into from
Aug 18, 2020

Conversation

odersky
Copy link
Contributor

@odersky odersky commented Aug 1, 2020

No description provided.

@nicolasstucki
Copy link
Contributor

@odersky I created #9534 to help unblock this PR

@odersky odersky force-pushed the drop-old-extension branch from 5d6b35b to 899c71a Compare August 16, 2020 10:14
@odersky odersky force-pushed the drop-old-extension branch from 899c71a to f61b4e9 Compare August 16, 2020 18:46
@odersky
Copy link
Contributor Author

odersky commented Aug 16, 2020

I give up. Can somebody else take this over please?

@nicolasstucki
Copy link
Contributor

I'll have a look at it

@odersky odersky force-pushed the drop-old-extension branch from 7990732 to ce9717b Compare August 17, 2020 08:38
@odersky
Copy link
Contributor Author

odersky commented Aug 17, 2020

@nicolasstucki I think I have switched branches by accident. Trying again.

@nicolasstucki
Copy link
Contributor

There was one extension left using the old syntax. Now it should be fine.

@odersky
Copy link
Contributor Author

odersky commented Aug 18, 2020

@nicolasstucki It seems something with the pushing of xml-interpolator went wrong. Did you push the new commits in that submodule.

@nicolasstucki
Copy link
Contributor

I pushed the commit and restarted the tests

@odersky
Copy link
Contributor Author

odersky commented Aug 18, 2020

Good to get this over the finish line!

@odersky odersky merged commit f46c030 into scala:master Aug 18, 2020
@odersky odersky deleted the drop-old-extension branch August 18, 2020 11:33
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