Open Zerwin opened 3 months ago
Seems like the build failed because the KSPASS variable was missing in the CI pipeline. Not really sure why though.
I'd like to delay merging this until we have rebased to a more recent version, as these changes might cause tricky to debug issues, for which one reason may be an outdated base.
Sounds good to me, as it mostly touches files that aren't usually changed in rebases it shouldn't be a problem to do this later.
Changes
Updates the dependencies, Gradle and build tools. Several linting warnings and errors are also fixed.
Checklist
I use the "bttv_" prefix for all resources I proposeWhen adding a string I also added it to thebttv.Res.strings
Enum andres/values/strings.xml
(inmod
) andres/values/public.xml
(indisass
)If my change is significant enough, I added it to the CHANGELOG.md undermaster
I'll add myself and everyone else who contributed to this change to the contributors list using all-contributorsI did not run ubi, because the build tools 33 use d8 instead of dx now but ubi itself seems to have dx hardcoded in some places.