Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lint(track_config): disallow difficulty 0 #447

Merged
merged 1 commit into from
Oct 22, 2021

Conversation

ee7
Copy link
Member

@ee7 ee7 commented Oct 22, 2021

We want this if exercism/docs#290 is merged.

Confirming that this is the only place we use difficulty:

$ git grep --heading 'difficulty'
src/lint/track_config.nim
95:      hasInteger(data, "difficulty", path, context, allowed = 1..10),
279:    # difficulty*: int

ErikSchierboom added a commit to ErikSchierboom/javascript that referenced this pull request Oct 22, 2021
ee7 pushed a commit to exercism/nim that referenced this pull request Oct 22, 2021
Previously, a difficulty value of 0 was allowed, and `configlet lint`
checked that each difficulty value was in the range from 0 to 10
inclusive.

However, a difficulty of 0 is now disallowed. Typically, exercises with
a difficulty value of 0 were deprecated exercises, but these still
appear on the website for users who started them before they were
deprecated, and so they should still have a difficulty level as normal.

With this commit, we change the difficulty of our one deprecated
exercise (`binary`) to an allowed value.

See:
- exercism/docs#290
- exercism/configlet#447
@ee7 ee7 merged commit 36a07d8 into exercism:main Oct 22, 2021
@ee7 ee7 deleted the lint-track-config-disallow-difficulty-0 branch October 22, 2021 11:35
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