Skip to content
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

Update jdk 17 #333

Closed
wants to merge 5 commits into from
Closed

Conversation

FatihYalmanbas
Copy link

@FatihYalmanbas FatihYalmanbas commented Feb 2, 2023

📌 References

🎩 What is the goal?

AGP 7.4.0 requires plugins to target JDK 11, update shot to enable environments running latest versions

Based on #292, rebased on master and ran tests

How is it being implemented?

Checkout original PR

How can it be tested?

  • Set your JDK to version 17+ (Can be done by settinh JAVA_HOME environment variable or through IDE settings)
  • Publish all artefacts to maven local
  • Run shot commands from a host application (shot-consumer projects can be used)

@FatihYalmanbas FatihYalmanbas marked this pull request as ready for review February 2, 2023 14:23
@@ -10,3 +10,5 @@ POM_LICENCE_NAME=The Apache Software License, Version 2.0
POM_LICENCE_URL=http://www.apache.org/licenses/LICENSE-2.0.txt
POM_DEVELOPER_ID=pedrovgs
POM_DEVELOPER_NAME=Pedro Vicente Gomez Sanchez

android.useAndroidX=true
Copy link
Author

Choose a reason for hiding this comment

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

This was required when running tests, there is probable other ways of fixing it. Didn't want to add more to this PR

When we don't have any new line at the end of properties
file and github workflow appends a RELEASE_SIGNING_ENABLED
flag at the end of it, it breaks builds. Add new line to file
and prepend it to github job.
@pedrovgs
Copy link
Owner

pedrovgs commented Jul 7, 2023

The update has been finally impemented here

@pedrovgs pedrovgs closed this Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants