Skip to content

Conversation

@skytin1004
Copy link
Collaborator

Purpose

Centralize and keep the “Other courses” section consistent across Microsoft Beginners series repos by managing it as a shared template in Co‑op Translator.

Reduce manual updates and ensure repos stay up-to-date whenever translations run.

Description

Implemented marker-based injection for “Other courses” in README.md

Related Issue

Does this introduce a breaking change?

When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.

  • Yes
  • No

Type of change

  • Bugfix
  • Feature
  • Code style update (e.g., formatting, local variables)
  • Refactoring (no functional or API changes)
  • Documentation content changes
  • Other... Please describe:

Checklist

Before submitting your pull request, please confirm the following:

  • I have thoroughly tested my changes: I confirm that I have run the code and manually tested all affected areas.
  • All existing tests pass: I have run all tests and confirmed that nothing is broken.
  • I have added new tests (if applicable): I have written tests that cover the new functionality introduced by my code changes.
  • I have followed the Co-op Translators coding conventions: My code adheres to the style guide and coding conventions outlined in the repository.
  • I have documented my changes (if applicable): I have updated the documentation to reflect the changes where necessary.

Additional context

- Add template: src/co_op_translator/templates/other_courses.md
- New markers: <!-- CO-OP TRANSLATOR OTHER COURSES START/END -->
- Utils: load_other_courses_template(), update_readme_other_courses()
- Generic marker replacer to support multiple shared blocks
- translate CLI: update Other courses before translation; echo only on success
- Keep languages table behavior as-is (updates when -l all)
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Oct 17, 2025
@skytin1004
Copy link
Collaborator Author

I have reviewed the changes and everything looks good.

@skytin1004 skytin1004 merged commit 4e8610c into Azure:main Oct 17, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant