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

Fix readme for new artifact name #433

Merged
merged 1 commit into from
Aug 19, 2021
Merged

Conversation

ptornhult
Copy link
Contributor

Was a bit confusing, but after checking the release note I managed to get the new dependency working without jcenter. Was just a small miss in the readme I think. Nice work!

@Sloy
Copy link
Member

Sloy commented Aug 19, 2021

Our bad! We too excited about the migration and copied the wrong name 😅

Thanks a lot for the fix, and sorry for the confusion ❤️

Sloy referenced this pull request Aug 19, 2021
@alorma alorma enabled auto-merge (squash) August 19, 2021 13:33
@alorma alorma merged commit 69c4623 into AdevintaSpain:master Aug 19, 2021
alorma pushed a commit that referenced this pull request Aug 26, 2021
Sloy added a commit that referenced this pull request Aug 27, 2021
* Update to latest Espresso version according to #430
-Update Espresso versions
-Add to the sample app Multidex configuration since there was a Dex error after updating the Espresso versions

* Fix readme for new artifact name (#433)

* Fix test errors, remove AndroidJUnit4 class runner, update sample app to minSdk=21 and remove Multidex from sample app

* Apply Barista code style to VisibilityAssertionsTest, optimize imports and fix a failing test on ChipTest

* Fix error on FailureHandlerTest assertion

Co-authored-by: Peter Törnhult <[email protected]>
Co-authored-by: Rafa Vázquez <[email protected]>
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