Ruby on Rails | Screencasts | Download | Documentation | Weblog | Community | Source

Ticket #9706 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

[PATCH] Various spelling fixes

Reported by: tarmo Assigned to: core
Priority: normal Milestone: 1.x
Component: ActionPack Version: edge
Severity: minor Keywords:
Cc:

Description

I ran the whole rails source through aspell, got some ~4400 words which aspell suspected of being spelling mistakes and then manually went through the list and grepped for the words that looked like actual mistakes. Most of the changes are to comments, most are for actual spelling errors, a couple may be a bit debatable, like hoze -> hose ;)

I'm assigning this to actionpack as there is no component that covers the whole rails source and actionpack seems to have the most changes in the patch.

Let me know if I should split the patch up or if this patch is a bad idea.

Attachments

various_spelling_fixes.patch (41.5 kB) - added by tarmo on 09/27/07 13:39:14.
various_spelling_fixes_2.patch (40.3 kB) - added by rmm5t on 09/27/07 15:27:05.
Re-based and one other spelling mistake.

Change History

09/27/07 13:39:14 changed by tarmo

  • attachment various_spelling_fixes.patch added.

09/27/07 15:26:13 changed by rmm5t

+1

I agree with doing this. I like the attention to detail, but I fear this patch is going to require a lot of re-basing unless it's applied soon.

I also found one other spelling mistake (convienence -> convenience). New re-based patch to follow.

09/27/07 15:27:05 changed by rmm5t

  • attachment various_spelling_fixes_2.patch added.

Re-based and one other spelling mistake.

09/27/07 15:30:28 changed by tarmo

I can easily rebase the patch on from time to time until it gets applied, it's in a git branch making rebasing very easy.

09/28/07 14:18:50 changed by david

  • status changed from new to closed.
  • resolution set to fixed.

(In [7666]) Fixed spelling errors (closes #9706) [tarmo/rmm5t]