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

Add new Kotlin dependencies #516

Closed
wants to merge 8 commits into from

Conversation

imashnake0
Copy link
Contributor

Hello/bonjour, I've added the new Kotlin dependencies from #498.

What?

Resolves #498.

Why?

To resolve #498.

How?

Added the four dependencies in #498.

Testing?

  • Tested all dependencies using sample-android and verified version keys in versions.properties.
  • Ran ./gradlew check.

Added some new Kotlin dependencies and their version aliases.
Ran the check task and tested on sample-android.
Added Kotlin deep learning dependencies and its version alias.
Ran check and tested using sample-android.
@imashnake0 imashnake0 changed the title New kotlin libs Add new Kotlin dependencies Mar 2, 2022
@imashnake0 imashnake0 marked this pull request as ready for review March 3, 2022 06:13
Copy link
Member

@LouisCAD LouisCAD left a comment

Choose a reason for hiding this comment

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

Hello, thanks for the PR!

I noticed that you're not using PascalCase as we do in other places for Dataframe and Deeplearning. Please use it.

Before committing your changes, you'll need to undo your edits to the following files:

  • dependencies-mapping-validated.txt
  • dependencies-versions-key-validated.txt
  • bundled-dependencies-validated.txt
    Once this is done, you can run the checks again so that they update the files properly, and then you can commit and push.

Changed case according to conventions.
Reran check.
@imashnake0 imashnake0 requested a review from LouisCAD March 3, 2022 13:55
@jmfayard
Copy link
Member

jmfayard commented Jul 2, 2022

Thanks @imashnake0
I've merged and improved your changes in #559

@jmfayard jmfayard closed this Jul 2, 2022
@imashnake0 imashnake0 deleted the new-kotlin-libs branch July 2, 2022 17:59
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.

Add new Kotlin dependency notations
3 participants