Skip to content
This repository has been archived by the owner on Jan 11, 2023. It is now read-only.

Remove async await #67

Merged
merged 2 commits into from
Jan 14, 2018
Merged

Remove async await #67

merged 2 commits into from
Jan 14, 2018

Conversation

EmilTholin
Copy link
Member

#62

lukeed
lukeed reviewed Jan 6, 2018
View reviewed changes
Copy link
Member

@lukeed lukeed left a comment

Choose a reason for hiding this comment

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

😆 I was about to begin the same PR. You beat me!

Haven't tested this locally, but seems to be about right at first glance while half-asleep in bed.

Travis seems to be having issues~

@EmilTholin
Copy link
Member Author

EmilTholin commented Jan 6, 2018 •
edited
Loading

Yeah, everything is working locally with node@6.12.3, but Travis is not too happy. It had some issues in my other PR as well, which had a few lines difference, so not quite sure what's wrong.

@Rich-Harris Rich-Harris merged commit 98e904d into sveltejs:master Jan 14, 2018
@EmilTholin EmilTholin deleted the remove_async_await branch January 14, 2018 10:53
@Rich-Harris Rich-Harris mentioned this pull request Jan 20, 2018
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@lukeed lukeed lukeed left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@EmilTholin @lukeed @Rich-Harris