-
Notifications
You must be signed in to change notification settings - Fork 111
Upgrade to build 0.9.0 #145
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
Conversation
natebosch
commented
Apr 25, 2017
- Replace declareOutputs with buildExtensions
- Upgrade build related dependencies
- Replace declareOutputs with buildExtensions - Upgrade build related dependencies
|
FYI, @natebosch – might could have updated the min SDK constraint to |
|
Min SDK constraint was already 1.21.1 |
|
Er 1.22.1
From mobile. Sorry for brevity and/or spelling.
…On Jun 14, 2017 4:48 AM, "Nate Bosch" ***@***.***> wrote:
Min SDK constraint was already 1.21.1
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#145 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AABCigOSc4gZWtghpwyXEpfIL4vJ773Hks5sD0oZgaJpZM4NH2ey>
.
|
|
Since the SDK constraint is tightened in package:build rather than by usage here I don't think it makes sense to update this constraint - we let pub figure that out for us. IMO it doesn't make sense to transitively search a package's dependencies to find the SDK constraint but to specify it based on language usage within this package. |
For me it's about support. If we can't test version X, we shouldn't claim to support it. |
- Replace declareOutputs with buildExtensions - Upgrade build related dependencies - Disable tests against sdk 1.21.1
- Replace declareOutputs with buildExtensions - Upgrade build related dependencies - Disable tests against sdk 1.21.1