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

generate: allow a concept slug to contain digits #664

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

ee7
Copy link
Member

@ee7 ee7 commented Sep 27, 2022

A concept slug is allowed to contain digits:

The concepts[].slug value must be a kebab-case string with length <= 255

[...]

kebab-case string: a string that [...] must match the regular expression: ^[a-z0-9]+(-[a-z0-9]+)*$

but configlet generate did not support such a slug.

With this commit, configlet generate now behaves as expected for an
introduction.md.tpl file like:

# Introduction

%{concept: basics-1}

Fixes: #663

A concept slug is allowed to contain digits [1], but `configlet generate`
did not support them.

With this commit, `configlet generate` now behaves as expected for an
`introduction.md.tpl` file like:

    # Introduction

    %{concept: basics-1}

[1] https://github.com/exercism/docs/blob/ba3eba34d334/building/configlet/lint.md#glossary

Closes: 663
@ee7 ee7 merged commit 1ab75f2 into exercism:main Oct 4, 2022
@ee7 ee7 deleted the generate-support-digits-in-concept-slug branch October 4, 2022 15:08
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.

Generate: slugs with numbers break generation
2 participants