-
Notifications
You must be signed in to change notification settings - Fork 6.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Proof P32-P35_P72-P75 #614 (PR 1/2) #782
Open
ngsctt
wants to merge
9
commits into
chrislgarry:master
Choose a base branch
from
ngsctt:P32-P35_P72-P75
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
http://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0618.jpg R0011 (L32) remove extra hyphen R0014 (L37) remove word "TIME" not in scan R0015 (L38) remove extra hyphen R0021 (L46) (un)correct typo in scan: "FLIGHT" → "FLIGNT" R0043 (L79) remove comma not in scan Comment out empty lines within comment blocks, for consistency
https://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0619.jpg R0045 (L82) add missing period R0054 (L94) correct "BY" → "ON" R0061 (L103) remove extra hyphen R0062 (L105) remove extra hyphen R0066 (L114) remove extra hyphen R0075 (L122) add missing hyphen R0077 (L126) remove word "THE" not in scan Remove line (L131) at end of page (after R0111) not reflected in scan Comment out empty lines within comment blocks, for consistency
https://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0620.jpg R0114 (L134) remove period not in scan Remove empty line (L137) not in scan (after R0116) R0137 (L145) remove extra hyphen R0138 (L146) remove extra hyphen R01395 (L154) remove extra hyphen R01396 (L155) remove extra hyphen R01402 (L162) correct "PROGRAM" → "PROGRAMS" R01404 (L165) remove extra hyphen R01406 (L167) remove extra hyphen R01408 (L169) remove extra hyphen Comment out empty lines within comment blocks, for consistency
https://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0624.jpg Add missing empty line (L325) after P1010 R1011 (L325) correct "LOC" → "LOG"; remove extra hyphen R1012 (L325) correct "WHITE, P." → "WHITE.P"; correct "DATE:" → "DATE"; correct "1 JUNE 67" → "1JUNE67" Remove empty line (L327) not in scan (after R1013) R1026 (L347) (un)correct "THEREFORE" → "THERFORE"; remove extra hyphen R1036 (L361) (un)correct "MINIMIZED" → "MINMIZED" Add missing empty line (L365) after R1038 R1039 (L368) remove space before tab
https://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0625.jpg R1052 (L386) remove period R1061 (L397) remove extra hyphen R1062 (L399) (un)correct "VEHICLE" → "VEGICLE"; remove extra hyphen R1066 (L404) remove extra hyphen R1069 (L408) remove extra hyphen R1075 (L416) replace period with hyphen R1111 (L424) remove extra hyphen
https://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0626.jpg R1112 (L426) remove extra hyphen R1137 (L437) remove extra hyphen R11394 (L444) correct "DELLVEET2" → "DELVEET2"; remove extra hyphen Remove empty line (L447) before R11402 not in scan R11404 (L450) remove extra hyphen R11405 (L451) remove period R11406 (L452) remove extra hyphen R11407 (L453) remove period R11408 (L454) remove extra hyphen; remove period R11409 (L455) remove period R1141 (L456) remove period
https://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0629.jpg P3800 (L572) correct "ADFLAG" → "AVFLAGA"
https://www.ibiblio.org/apollo/ScansForConversion/Luminary099/0630.jpg P4600 (L598) add spaces to match alignment in scan R4652 (L603) added space to reflect alignment in scan 47024 (L612) left as "SXA,1" not "SXA.1"
Asterisks in P3800, P4600, P4800 corrected to `.`, per chrislgarry#782 (comment)
Fixed in 754153e 🙂 |
I've added the hacktoberfest-accepted label to ensure your contribution counts towards Hacktoberfest, as I haven't been able to review as many PRs as I had hoped over the past few days during my free time. PRs are being throughly reviewed from oldest to newest 👍 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR 1 of 2, partially resolves #614, covering pages 0618–0631
I've left the indentation as it was in the file, using tabs instead of spaces. I've also commented out empty lines within comment blocks earlier in the file, to be consistent with the rest of the comment blocks.
I've also left the
•
characters as*
(P3800, P4600, P4800).Corrections made: