Closed (fixed)
Project:
Drupal core
Version:
8.0.x-dev
Component:
documentation
Priority:
Minor
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
30 Oct 2011 at 14:03 UTC
Updated:
29 Jul 2014 at 20:05 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
xjmComment #2
xjmThis is probably unobtrusive enough to backport.
Comment #3
xjmComment #4
xjmComment #5
xjmActually, this would make a good novice issue.
Note that any patch for this issue will need to be rerolled once #1315886: Clean up API docs for includes directory, files starting with A-C is fixed, so probably best to wait until then.
Comment #6
ryan.ryan commentedI can do this one :)
Comment #7
ryan.ryan commentedOkay, this is my first time creating and uploading a patch - I couldn't see a double-space after a period in the constructor documentation. If I missed it, I can fix it. Wow, core office hours really are a great way to both help and learn. Now I can see a patch and understand it much better. I thought all of the patch info was typed manually. GIT really does the hard work.
Comment #8
tregeagle commentedHey Ryan,
Nice one. The patch applies cleanly and it looks like all the typos are fixed.
Now I just need to find my first patch to roll...
Comment #9
webchickNot sure why the testbot didn't report back, but the tests passed.
Committed and pushed to 8.x and 7.x. Thanks! Congrats on your first core patch, Ryan! :D
Comment #10
jthorson commented#7: clean-up-minor-code-1326482-7.patch queued for re-testing.Sorry ... just troubleshooting testbots.
Comment #12
jthorson commentedComment #13.0
(not verified) commentedAdded clarification.