Skip to content

no concept practiced more than 10 times - #797

Merged
ryanplusplus merged 1 commit into
exercism:mainfrom
keiravillekode:practices
Nov 12, 2026
Merged

ryanplusplus merged 1 commit into
exercism:mainfrom
keiravillekode:practices

Conversation

@keiravillekode

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions

Copy link
Copy Markdown
Contributor

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)

@keiravillekode

Copy link
Copy Markdown
Contributor Author

Context: these warnings came up when preparing a configlet release - see forum

@ryanplusplus

Copy link
Copy Markdown
Member

What's the intent of this check and this change? Are we trying to make each concept be more focused with fewer related exercises?

@keiravillekode

Copy link
Copy Markdown
Contributor Author

The intent with the change is to address the warnings reported by bin/configlet lint

Warning: The Concept ... appears ... times in the practices arrays of user-facing Practice Exercises, but can only appear at most 10 times:
.../crystal/config.json

The checks were added in September 2021. There was the suggestion that the warnings might become errors at the end of October.

lint_20261112.txt

@ryanplusplus
ryanplusplus merged commit cb2a7d3 into exercism:main Nov 12, 2026
15 checks passed
@keiravillekode
keiravillekode deleted the practices branch November 13, 2026 00:56
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