Skip to content

Updated dependencies#15

Open
fobo66 wants to merge 13 commits intoZacSweers:mainfrom
fobo66:update
Open

Updated dependencies#15
fobo66 wants to merge 13 commits intoZacSweers:mainfrom
fobo66:update

Conversation

@fobo66
Copy link
Copy Markdown

@fobo66 fobo66 commented Aug 28, 2022

I took a liberty to update versions of everything. Some Gradle plugins broke (e.g. Shadow doesn't seem to work, but I'm not sure about it) or got obsolete, so I removed them.

Unfortunately, for some reason Studio didn't want to see Android benchmarks, so I had to extract them to the top level module, hope that's OK.

As an extra, I've added some useful Gradle properties to speed up builds. Not sure if they can affect benchmark results, most likely not, but it can improve build times at least.

@manishindiainc
Copy link
Copy Markdown

Any reason for not merging this to main?

@ZacSweers
Copy link
Copy Markdown
Owner

This isn't a project I actively maintain or check, sorry. I'll take a look soon. Please don't assume it's been intentionally ignored, open source is usually spare time and things slip :)

@fobo66
Copy link
Copy Markdown
Author

fobo66 commented Feb 1, 2023

I was unable to update Kotlin to 1.8, it didn't work for some reason: it was complaining on the function with empty body for serializer(). Probably will look into it in the separate PR

@fobo66
Copy link
Copy Markdown
Author

fobo66 commented Feb 4, 2023

Kotlin update on 1.8.10 worked, luckily. There was a bug in compiler that was causing the issue

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