-
Notifications
You must be signed in to change notification settings - Fork 553
Find a way to DRY up the IOApp
s
#4333
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
Comments
Do you mind if I take a crack at this? |
Go for it! Make sure to double-check |
@kapunga @djspiewak maybe should wait until #4201 is merged, since that changes |
I've started working on a draft PR. It's got a ways to go and I'm happy with the approach I'm taking, but if you are super opinionated, you may want to have a look-see. |
Somewhat of a follow up from #4201, though this has been a problem for a long time.
Basically all of the
IOApp
s are very similar (especially JVM and Native), with just a few differences sprinkled throughout, some different documentation, and a few platform-specific APIs. We should try really hard to consolidate the actual code if we can because it's starting to get hard to maintain, especially between JVM and Native.The text was updated successfully, but these errors were encountered: