Closed Bug 142792 Opened 23 years ago Closed 23 years ago

Output from checksetup.pl contains misspelled words

Categories

(Bugzilla :: Installation & Upgrading, defect, P1)

2.15
defect

Tracking

()

RESOLVED FIXED
Bugzilla 2.16

People

(Reporter: ddkilzer, Assigned: ddkilzer)

Details

Attachments

(1 file)

The patch for Bug 140355 added some output that the user will see under certain circumstances. It contains a couple misspellings. Also changed "Perl" to "Perl code" and "as the user" to "as the user that". Patch to follow shortly.
Status: NEW → ASSIGNED
Keywords: patch, review
Priority: -- → P1
Target Milestone: --- → Bugzilla 2.16
Attached patch Patch v.1Splinter Review
Fixes misspellings of "databasa" and "arbitary". Adds "code" and "that" to make sentence clearer.
Comment on attachment 82649 [details] [diff] [review] Patch v.1 2xr=gerv. Gerv
Attachment #82649 - Flags: review+
Fixed. Checking in checksetup.pl; /cvsroot/mozilla/webtools/bugzilla/checksetup.pl,v <-- checksetup.pl new revision: 1.144; previous revision: 1.143 done Gerv
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
It strikes me misspellings like these would be a good thing to add to the testing suite.
AU or US spelling? :) Besides spell checkers don't like technical stuff, even if we did just check comments, we'd still be mentioning variable names and the like.
AU spelling of course. I didn't mention hooking up a spell checker, just looking for specific misspellings. I don't see a problem with checking variable names.
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: