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 Google accompanist modules #556

Merged
merged 3 commits into from
Aug 31, 2022

Conversation

rlatapy-luna
Copy link
Contributor

No description provided.

@ghost
Copy link

ghost commented Jun 28, 2022

Review on Crocodile

@LouisCAD
Copy link
Member

Hello, thanks for the PR.

Why did you revert changes to test resources?

@rlatapy-luna
Copy link
Contributor Author

Hello, thanks for the PR.

Why did you revert changes to test resources?

Hello, I thought it was breaking the test 🤔

@jmfayard
Copy link
Member

jmfayard commented Jul 2, 2022

@rlatapy-luna nope the tests are here to prevent you from typos and so on.

It seems that there are more libraries than what your PR contains.
https://mvnrepository.com/search?q=com.google.accompanist

Can you add them too and run the tests please?

@rlatapy-luna rlatapy-luna changed the title Add Google accompanist-placeholder Update Google accompanist modules Jul 4, 2022
@rlatapy-luna
Copy link
Contributor Author

Can you add them too and run the tests please?

I added missing accompanist lib and removed deprecated ones (glide & imageloadingCore)

@LouisCAD
Copy link
Member

Thanks for updating!
I'll review it soon.

@LouisCAD LouisCAD self-requested a review August 26, 2022 22:40
@LouisCAD LouisCAD self-assigned this Aug 26, 2022
@pull-request-size pull-request-size bot added size/L and removed size/M labels Aug 31, 2022
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.

It was nicely done, thank you!

I had to rebase onto main and redo the removals tracking because I did further changes before taking on to this PR, should have merged this first 😅

@LouisCAD LouisCAD merged commit 6bb8dce into Splitties:main Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants