docs: restore fixes lost in a conflicted merge, and make CI watch main itself - #80
Merged
ZhuYaoHui1998 merged 2 commits intoSep 7, 2026
Conversation
Merge commit 11f09f7 resolved README conflicts in favour of a local branch that predated Seeed-Projects#58, Seeed-Projects#61 and Seeed-Projects#63, reverting all of their README changes. The same merge dropped LAN-GER's community row for the Isaac Sim-to-Real VLA course (af00e42), whose GIF is still in community/ with nothing pointing at it. Nothing was force-pushed — the commits are all still ancestors of main; the content was resolved away by hand across five large files, which leaves no obvious trace. Restored, by re-applying onto current main rather than reverting the merge, so the newer work in it survives (the DLI course link, the Star Arm 102 purchase links, the isaacsim wiki link): - Feishu redirect back to the AliExpress product (4 files) - Sensor/IMU 404 back to Sensor_accelerometer (5 files) - shields.io badges re-escaped: the literal space and the raw emoji - readme.md/#about-power-supply trailing slash (4 files) - RS BOM links back to README.md, which is the real filename (2 files) - zh: reach back to 767 mm, reSpeaker row restored, RS Isaac Sim back to completed; fr and jp likewise - jp: the RS roadmap table back to Japanese from the French copy - LAN-GER's VLA course row, in each language's own wording Neither check could have caught this: both only run on pull_request, and these commits were pushed straight to main. link-check would have waited for its Monday schedule; readme-sync has no schedule, so it would never have run at all. Both now also run on push to main, and readme-sync gets the same weekly schedule link-check has. Closes Seeed-Projects#79 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This was referenced Sep 8, 2026
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.
Closes #79
What happened
Merge commit
11f09f7("Merge branch 'main' of …", 2026-09-03) resolved README conflicts in favour of a local branch that predated #58, #61 and #63, and reverted all of their README changes. The same merge dropped @LAN-GER's community row for the Isaac Sim-to-Real VLA course (af00e42) — its GIF is still sitting incommunity/with nothing pointing at it.Nothing was force-pushed; every original commit is still an ancestor of
main. The content was simply resolved away by hand across five large README files, which leaves no trace in the log. This is an easy mistake to make and not really avoidable by care alone — which is why the second half of this PR matters more than the first.What this restores
Re-applied onto current
mainrather than by reverting11f09f7, so the legitimate newer work in that merge survives — the DLI course link, the Star Arm 102 purchase links, therebot_arm_b601_dm_isaacsimwiki link are all left as they are.Sensor/IMU404 →Sensor_accelerometerreadme.md/#about-power-supplytrailing slashREADME.md(the real filename)The Chinese Isaac Sim row keeps the newer
DLI Courselink while its status returns to ✅ — newer work preserved, the fix restored.The part that stops it recurring
Neither check could have caught this. Both ran on
pull_requestonly, and these commits were pushed straight tomain:link-check.yml—pull_request+ weeklyschedule. It would have caught the broken links on Monday, four days late.readme-sync.yml—pull_requestonly, no schedule at all. It would never have run.That is a gap in what I set up in #58 and #63, not a process failure by anyone. A guard that only watches pull requests does not protect a repository whose maintainers also push directly. Both workflows now also run on
pushtomain, andreadme-sync.ymlgets the same weekly schedulelink-check.ymlhas.Verification
tools/check_readme_sync.py— 5 disagreements on currentmain, 0 on this branch.main, 0 here.actionlintclean on both workflows; the parsed trigger lists confirmpush → branches: [main]on each..lycheeignore, 0 broken.Two notes
This is why #78 is red. PR #78 is failing its link check on these same reverted links, not on anything it changes. Merging this turns it green — that red is the check working correctly, on a real regression.
One duplicate to decide on. The VLA course URL now appears twice in
README.md: once in the restored community row, once in the roadmap'sDLI Courselink added afterwards byfa2ef02. Both are legitimate in their own section and they never coexisted before, so I have left both. Say the word if you would rather keep only one.🤖 Generated with Claude Code