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 dependencies #54

Merged
merged 5 commits into from
Jan 17, 2023
Merged

Update dependencies #54

merged 5 commits into from
Jan 17, 2023

Conversation

sproctor
Copy link
Contributor

compose.material is forced as a dependency on consumers. Nothing from material is exposed in the API, so this should be an implementation instead of api dependency.

Copy link
Collaborator

@ffgiraldez ffgiraldez left a comment

Choose a reason for hiding this comment

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

please run ./gradlew spotlessApply and fix formatting issues

@sproctor
Copy link
Contributor Author

Sorry about that. IntelliJ likes to merge imports into wildcards.

@ffgiraldez ffgiraldez merged commit 3ab6b14 into godaddy:main Jan 17, 2023
@sproctor sproctor deleted the remove_api branch February 1, 2024 15:05
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