Talk:Android Debug Bridge/GA1

GA Review

Article (edit | visual edit | history) · Article talk (edit | history) · Watch

Reviewer: Rusalkii (talk · contribs) 18:24, 30 November 2021 (UTC)[reply]

I'm a first time reviewer, so I intend to ask for a more experienced review to look this over once the review is complete.

Initial thoughts: the article seems sparse for a GA. No illustrations but the one in the infobox, at this size I would want one or two more; perhaps an image of ADB in use? Architecture section is two sentences and should be significantly expanded or merged elsewhere, preferably expanded. Rusalkii (talk) 18:24, 30 November 2021 (UTC)[reply]

GA review (see here for what the criteria are, and here for what they are not)

It is reasonably well written.
a (prose, spelling, and grammar): b (MoS for lead, layout, word choice, fiction, and lists):
It is factually accurate and verifiable.
a (reference section): b (citations to reliable sources): c (OR): d (copyvio and plagiarism):

Copyvio: Earwig gives 6.5% similarity, with a couple sentences in the security section resembling this source. Could be rewritten but seems okay. Rusalkii (talk) 20:43, 30 November 2021 (UTC)[reply]

It is broad in its coverage.
a (major aspects): b (focused):
It follows the neutral point of view policy.
Fair representation without bias:
No bias or promotional tone. Security issues given reasonable weight.
It is stable.
No edit wars, etc.:
Recent history since August split from Android software development#Android Debug Bridge (ADB) has been almost entirely steady improvements by nominator PhotographyEdits, with a single incident of minor vandalism. Rusalkii (talk) 18:41, 30 November 2021 (UTC)[reply]
It is illustrated by images and other media, where possible and appropriate.
a (images are tagged and non-free content have fair use rationales): b (appropriate use with suitable captions):
Own work screenshot and public domain logo. Could do with more images, but I couldn't find any in commons or appropriately licences after a quick search. Rusalkii (talk) 20:43, 30 November 2021 (UTC)[reply]
Overall:
Pass/Fail:

Comments

Lead

I'm seeing both adb and ADB in sources, but article should not use both in prose. Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
It consists of a client and server on the host PC, where the server connects to the daemon on the Android device. "It consists of" is awkward and "where the server..." is confusing. This sentence feels like it wants to be several sentences explaining how ADB works in slightly more depth. Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
available since 2007 available where? to whom? was it created before then? Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
Why are the shell and backups the features mentioned in the lead? Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
It has seen different security attacks and improvements to mitigate these This reads very vaguely. Why are these prominent enough for the lead? Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
Images in infobox: is there an ADB specific logo that could be added? OK if not. Rusalkii (talk) 20:43, 30 November 2021 (UTC)[reply]
Overall impression of lead: vague. There's still plenty of space before the lead gets too long, it would be good to see some more concrete descriptions of what it is, where it came from, and how it is used. Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]

Features

It isn't clear to me what's core and what's peripheral. I would format this by starting with the core goal of ADB (presumably mobile debugging from a connected computer), explaining the central features and how they're used for that, and then mentioning any other features. Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
Given the length of this section, the name of the file backups are saved to doesn't seem relevant. Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
Some graphical interfaces have been made available. Weasel words - "some"? made available how? by whom? Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
This section could use an image of the GUI. Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]

History

I would rename this section Development. Rusalkii (talk) 20:43, 30 November 2021 (UTC)[reply]
Reads like a list. There's some attempt at going chronologically, but 2015 follows 2017 and version numbers are interlaced with years seemingly interchangeably. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
first beta release of the Android SDK clarify that ADB used to be part of Android SDK in this sentence rather than the next, otherwise it reads like a nonsequitor. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
Microsoft released an Android emulator that is able to connect using ADB Connect to what? Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
In Android 12 the adb backup command will be limited. what's the current version? May want to add to infobox as well. Also, limited to what/how? Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]

Enablement

Section heading is confusing. Maybe "Setup"? Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
Host computer section covers Windows and Ubuntu/Debian, but not Mac or any other Linux system. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
The procedure does differ for some Android vendors. Either delete this whole sentence or specify which. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
It has been recommended to add the folder containing the binaries to the PATH environment variable, it has been recommended to also install the android-sdk-platform-tools-common package by whom? Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
(API level 17) is an API "level" a release version in this context? clarify or wikilink if possible. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
In this section (and others), article goes back and forth between "device" and "phone". I assume it can be used on e.g. tablets and so device is more appropriate. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]

Architecture

Nothing wrong with the content, but it seems incomplete as a standalone section. Rusalkii (talk) 23:56, 30 November 2021 (UTC)[reply]

Security

I added a paragraph to divide RageAgainstTheCage and the unnamed next vulnerability. It's unclear if the new second paragraph describes one vulnerability or several. Clarify and/or split into paragraphs. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]

General

I made some minor prose/formatting/grammar changes. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
Some important topics that seem to be missing
- Who made ADB? Who's developing it? It's mentioned in the infobox and the implication in Google made it possible, but this deserves some content in the article, ideally both in the lead and the history section. Rusalkii (talk) 19:24, 30 November 2021 (UTC)[reply]
- How and what is it used for? I see sources talking about both debugging, as implied by the title, and modding. The article also mentions malware removal in the security section, but I'm not sure it belongs there- that seems to be about security issues with ADB, not vice versa. 23:44, 30 November 2021 (UTC)

Sources

All nontrivial statements are cited, no sources in lead. Rusalkii (talk) 23:44, 30 November 2021 (UTC)[reply]
Sources look generally okay. I'm not thrilled about the number of how to articles but none of them seem individually objectionable. Rusalkii (talk)
Spotcheck that claims match sources:
- 7 (Macworld) does not say that the 2007 release was a beta version.
- 8 (Android Police) looks good.
- 9 (Softpedia) looks good.
- 18 (packages.debian.org) looks good.
- 20 (Tech Republic) looks good.
- 24 (Ars Technica) looks good.
- 25 (ZDNet) is perhaps too closely paraphrased but otherwise good.