Add two Visa new grad OA practice questions - #153
Draft
tomtoto757 wants to merge 1 commit into
Draft
Conversation
This file contains hidden or 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
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.
Two published Visa coding exercises are missing from the question bank. This adds their canonical FastPrep titles and practice links, with the latest reported date of Aug 31, 2026:
The public metadata identifies both as Visa new-grad OA problems. The README and Coding page now contain 2,141 and 1,809 rows respectively. All 2,139 prior README rows, 1,807 prior Coding rows, their relative order, and the company list are unchanged.
README.md remains the authored source. The generated formats/coding.md change is required by the current CONTRIBUTING.md workflow and keeps the linked Coding page consistent. This PR changes only those two Markdown files; it includes no problem statements, solutions, source images, runtime code, or workflow changes.
Verification:
python3 scripts/refresh-freshness.pypasses and is idempotent: zero marker changes and zero reordered positions on the final run.python3 scripts/sync-practice-formats.pyandpython3 scripts/sync-practice-formats.py --checkpass against the current public catalogs.PYTHONDONTWRITEBYTECODE=1 python3 -m unittest scripts/test_sync_practice_formats.pypasses all 11 tests.git diff --check origin/master...HEADpasses.An independent Codex reviewer,
source_review, reviewed final headaf8001ae1005ce26c9b9f7b82d749842ed196d21against current masterf3e590c5989831ef597def6d1d8797c34d717b38. The review covered the complete diff, native contribution contract, generator and hourly consumer, all-row invariants, published metadata, canonical routes, and publication eligibility. The reviewer independently reran the native check, all 11 tests, full table verification, public HTTP checks, freshness idempotence, and whitespace validation. No findings remain.The native checker still reports 158 pre-existing legacy routes absent from the public catalog; none is introduced by this change. README is 495,300 bytes, below the native 500,000-byte limit. Application type checking and builds are not applicable to this Markdown-only change. This is a draft PR; no merge or deployment is requested.