Skip to content

Conversation

@MojtabaHs
Copy link
Contributor

@MojtabaHs MojtabaHs commented Jun 4, 2025

All changes are fully backward compatible.

So we can add [no important files changed] to merge-commit message.

@github-actions
Copy link
Contributor

github-actions bot commented Jun 4, 2025

Hello 👋 Thanks for your PR.

This repo does not currently have dedicated maintainers. Our cross-track maintainers team will attempt to review and merge your PR, but it will likely take longer for your PR to be reviewed.

If you enjoy contributing to Exercism and have a track-record of doing so successfully, you might like to become an Exercism maintainer for this track.

Please feel free to ask any questions, or chat to us about anything to do with this PR or the reviewing process on the Exercism forum.

(cc @exercism/cross-track-maintainers)

@meatball133
Copy link
Member

The introduction docs for concept exercises also execist under concepts/concept/about.md and introduction,md. In this case they are located under the concept for-loops. Both of which has to be updated.

For the Swift track to keep things simple have I made so the introduction docs for the exercise is the same as for the concept (unless an exercise have many concepts). The about file might differ and might have some extra headings the introduction one don't have.

@MojtabaHs
Copy link
Contributor Author

I have fixed them as requested. I think we should make a single source of truth for all identical documents. Maybe using links 🤷🏻‍♂️

@meatball133
Copy link
Member

There is this tool which is part of configlet (the cross-track Exercism tool): https://exercism.org/docs/building/tracks/concept-exercises#h-file-docs-introduction-md-tpl. However, it only allows the introduction to be templated. Not the about file.

Alternatively, you can develop a custom solution (when becoming maintainer). I have done such a thing on the main track I maintain: https://github.com/exercism/crystal/tree/main/document-generator, I haven't worked on it for a while but it does work.

@meatball133 meatball133 merged commit 0513250 into exercism:main Jun 4, 2025
9 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants