-
Notifications
You must be signed in to change notification settings - Fork 5.3k
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
Gradle configuration #962
Comments
I'd second that (though I'm quite late to the party). Out of the box Kotlin support would be nice, but what would be better is if the gradle config could be changed to not use deprecated features? I had half of my branches start refusing to build when I updated gradle. |
Yea gradle is a pain in the ass 90% of the time; you have to align all the versions like stars to get your code to build 😅 |
I made a PR with tons of fixes and updates including this one and got it instantly closed by a bot with a message saying "sorry we don't take prs" 😭 |
That's because the FtcRobotController repository is the "output" of our internal build process, it's actually automatically generated. We cannot accept PRs to the repo because it cannot go backwards. If you have suggestions for updates, you can send a link to your PR to [email protected] and we'll consider it. -Danny |
Welp fix the build process to not use things that are years behind please? |
Comments like this are not helpful. A lot of things you depend on no longer work properly in newer versions of Gradle, and there are multitudes of build processes that you're not aware of that must be retrofitted because they also break in newer versions. Also, with the planned obsolescence of the FTC SDK build environment there's only so much work we're going to do - the ROI just doesn't make sense. |
Hello, I'd like to propose a few changes towards the gradle configuration file:
compileSdkVersion
is upgraded from29
to34
The primary reason for this request is out of the box support for Kotlin, however keeping deps up to date is good practice in general. I could make a PR but I'm too lazy :P
The text was updated successfully, but these errors were encountered: