Add latest ByteDance CodeSignal OA questions - #151
Closed
tomtoto757 wants to merge 1 commit into
Closed
Conversation
Collaborator
Author
|
Superseded by consolidated batch PR #150 at exact reviewed head 3a31217. PR #150 preserves all four ByteDance rows byte-for-byte alongside the three Amazon updates; independent final-SHA review confirmed no content loss. #150 remains draft only for the inherited generated-format synchronization debt. |
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.
Summary
(for mle also :)suffixREADME.mdVerification
python3 scripts/refresh-freshness.py— second pass was idempotent with 0 marker changes and 0 reordered rowsPYTHONDONTWRITEBYTECODE=1 python3 -m unittest -v scripts/test_sync_practice_formats.py— 10/10 passed🔥 Aug 25, 2026git diff --check— passedConcurrent README ownership
Open draft PRs #147–#150 also touch
README.md, but none contains or owns any of these four ByteDance routes. This focused branch starts from the latestmaster(3c98ad57c68670b3cfb001a50c7818d470548683) and intentionally does not absorb their unrelated IBM, Anthropic, Walmart, or Amazon changes.Independent regression review
An independent read-only reviewer checked exact base
3c98ad57c68670b3cfb001a50c7818d470548683and exact headd0344d2ad2a78676268987da7601d7df01a0ce18; no PR-introduced findings.🔥 → 🆕and 67🆕 → unbadgedfreshness changesgit diff --check; all passed and the reviewed head remained byte-stablemasterand the PR branch still equal the reviewed SHAs, and PR Add latest ByteDance CodeSignal OA questions #151 is draft, clean, and mergeableDraft status and remaining risk
python3 scripts/sync-practice-formats.py --checkreports four stale generated format pages on both the exact base and this head. Both runs report the same four pages, 148 catalog-missing routes, and zero unclassified rows. The generated README Coding label remains 1,772 while the exact base contains 1,777 Coding rows and this head contains 1,781, an increase of exactly the four intended additions. The inherited company summary likewise remains unchanged at214+/+204while both SHAs contain the same 215 unique sorted company names; ByteDance already occurs exactly once.Those counters are owned by
scripts/sync-practice-formats.py, which regenerates the README counter and allformats/*.mdpages together. Manually changing only the README counter would leave the generated Coding page inconsistent, while running the full generator would absorb unrelated live-catalog drift beyond this focused README-only Stage 4 scope. The required row freshness validator passes every row and is idempotent; the generated-count drift is therefore documented rather than partially rewritten here.The PR remains draft under the Stage 4 default pending maintainer review.
Expected runtime impact: none; this changes README catalog metadata and deterministic freshness markers only.