Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

Jwen/global check #370

Merged
merged 35 commits into from
Jun 9, 2022
Merged

Jwen/global check #370

merged 35 commits into from
Jun 9, 2022

Conversation

jiabaow
Copy link
Collaborator

@jiabaow jiabaow commented Jun 9, 2022

No description provided.

@jiabaow jiabaow self-assigned this Jun 9, 2022
/* the array must be declared explicitly (and not with buzzersToRows.keys)
else the buzzers may not be ordered properly
*/
private val buzIds = arrayListOf(R.id.buzzer_0, R.id.buzzer_1, R.id.buzzer_2, R.id.buzzer_3)
Copy link

Choose a reason for hiding this comment

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

Similar blocks of code found in 3 locations. Consider refactoring.

}
}

/**
Copy link

Choose a reason for hiding this comment

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

Identical blocks of code found in 2 locations. Consider refactoring.

@codeclimate
Copy link

codeclimate bot commented Jun 9, 2022

Code Climate has analyzed commit 35deb53 and detected 3 issues on this pull request.

Here's the issue category breakdown:

Category Count
Duplication 3

The test coverage on the diff in this pull request is 96.9% (80% is the threshold).

This pull request will bring the total coverage in the repository to 84.3% (0.5% change).

View more on Code Climate.

@jiabaow jiabaow marked this pull request as ready for review June 9, 2022 14:30
@MaximeZmt MaximeZmt self-requested a review June 9, 2022 14:31
Copy link
Collaborator

@laurislopata laurislopata left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Owner

@MaximeZmt MaximeZmt left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@MaximeZmt MaximeZmt merged commit 626dc9c into main Jun 9, 2022
@MaximeZmt MaximeZmt deleted the jwen/search_user_improve branch June 9, 2022 14:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants