Skip to content

Fix 4.1.0 lint #1329

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

Merged
merged 4 commits into from
Jun 6, 2018
Merged

Fix 4.1.0 lint #1329

merged 4 commits into from
Jun 6, 2018

Conversation

samtstern
Copy link
Contributor

No description provided.

Change-Id: I1929e199d1676ae26e8e84fb528d8846a740b911
@samtstern samtstern changed the base branch from master to version-4.1.0-dev May 31, 2018 16:48
@firebase firebase deleted a comment from googlebot May 31, 2018
@samtstern
Copy link
Contributor Author

@SUPERCILEX now that quality.gradle is gone where should I declare that we want to ignore the MissingTranslation for now. In the baseline?

@@ -79,7 +79,7 @@ protected void onFailure(@NonNull Exception e) {
protected void onSuccess(@NonNull PhoneVerification verification) {
if (verification.isAutoVerified()) {
Toast.makeText(
PhoneActivity.this, R.string.fui_verified, Toast.LENGTH_LONG).show();
PhoneActivity.this, R.string.fui_auto_verified, Toast.LENGTH_LONG).show();
Copy link
Collaborator

Choose a reason for hiding this comment

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

Whoops, thanks for catching that! 😊

@SUPERCILEX
Copy link
Collaborator

SUPERCILEX commented May 31, 2018

@samtstern right here:

disable("UnknownNullness") // TODO fix in future PR
Or in the module's build file like here:
disable("UnusedQuantity")

@SUPERCILEX
Copy link
Collaborator

Looks like lint didn't like your baseline changes. Would probably just be easier to move the lambda around. 🤷‍♂️

@SUPERCILEX SUPERCILEX mentioned this pull request Jun 2, 2018
samtstern added 3 commits June 6, 2018 09:46
Change-Id: I2317eb13e18e5dd837f8cb45c146f16d4676b2d5
Change-Id: I531edc2b32679a9c73e519586f448aa9d63fab33
Change-Id: Ie4627aab27ebaf0f8e03100635e980c0a08e7d01
@samtstern samtstern merged commit 44b1a4f into version-4.1.0-dev Jun 6, 2018
@samtstern samtstern deleted the fix-4.1.0-lint branch June 6, 2018 17:29
@samtstern
Copy link
Contributor Author

WHY DOES SQUASH AND MERGE NOT WORK

@samtstern
Copy link
Contributor Author

image

@SUPERCILEX
Copy link
Collaborator

SUPERCILEX commented Jun 6, 2018

Lol, report it? 😭

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