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

[Internal-testutils] Migrate gradle files to Kotlin DSL #1599

Merged
merged 2 commits into from
May 3, 2023

Conversation

oas004
Copy link
Contributor

@oas004 oas004 commented May 1, 2023

Following up #1578 and relating to #1579

Migrating the :internal-testutils module to Kotlin DSL.

I have split up the PR into two commits. The first one prepares the file to be migrated by using steps described in these docs. The second one is where I actually convert the file to .kts file and updates the rest of the file to be compatible with Kotlin DSL.

Testing

The testing I have done for this is to check that it syncs and builds the sample application. Let me know if there is any other test than what the CI runs that you want me to run as well.

@bentrengrove bentrengrove self-requested a review May 3, 2023 03:23
Copy link
Collaborator

@bentrengrove bentrengrove left a comment

Choose a reason for hiding this comment

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

Thank you for all of these! It is super appreciated!

@bentrengrove bentrengrove merged commit 843e648 into google:main May 3, 2023
@oas004 oas004 deleted the feature/internal-testutils-kts branch May 4, 2023 06:18
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.

2 participants