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

Dispose ApplicationEngine after using #501

Merged
merged 1 commit into from
Dec 8, 2019
Merged

Dispose ApplicationEngine after using #501

merged 1 commit into from
Dec 8, 2019

Conversation

erikzhang
Copy link
Member

No description provided.

@erikzhang erikzhang mentioned this pull request Dec 6, 2019
Closed
shargon
shargon approved these changes Dec 6, 2019
View reviewed changes
Copy link
Member

@shargon shargon left a comment

Choose a reason for hiding this comment

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

I prefer the old fashioned way (with {}) hahha but it's ok for me

@erikzhang erikzhang merged commit 1cd57b1 into master Dec 8, 2019
@erikzhang erikzhang deleted the fixes/dispose branch December 8, 2019 14:15
@cloud8little cloud8little mentioned this pull request Dec 9, 2019
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@shargon shargon shargon approved these changes

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@erikzhang @shargon