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

🤖 Add file patterns to config.json #104

Merged
merged 2 commits into from
Mar 19, 2021

Conversation

ErikSchierboom
Copy link
Member

âš  This PR requires you to make a simple change before merging. âš 


To save maintainers from having to manually specify the files key in their exercises' .meta/config.json files, we are providing support for track-level patterns. See this PR for details.

This PR adds (purposefully wrong) file patterns to the config.json file. It is up to you, the track maintainers, to change these patterns to their correct value.

You can use the following placeholders:

  • %{kebab_slug}: the kebab-case exercise slug (e.g. bit-manipulation)
  • %{snake_slug}: the snake_case exercise slug (e.g. bit_manipulation)
  • %{camel_slug}: the camelCase exercise slug (e.g. bitManipulation)
  • %{pascal_slug}: the PascalCase exercise slug (e.g. BitManipulation)

We will soon update configlet to enable it to automatically populate the .meta/config.json file's files property, at which point we will then batch-PR updates to all tracks that have merged this PR.

Tracking

exercism/v3-launch#19

@ErikSchierboom ErikSchierboom added the v3-migration 🤖 Preparing for Exercism v3 label Feb 17, 2021
There is currently no exemplar and example files, so using the same
example file for both.
@ErikSchierboom
Copy link
Member Author

@kotp I'll also need an approval to be able to merge this. Do you have permissions to do so?

@kotp
Copy link
Member

kotp commented Mar 18, 2021

And I don't for VimL/Vimscript either.

@ErikSchierboom
Copy link
Member Author

@kotp You should have permissions now. It does still fail to register your review as an approving review. Could you maybe re-approve?

@kotp
Copy link
Member

kotp commented Mar 18, 2021

Checks have failed, so probably now stopped by that. (I do have "close with comment" permissions now though.)

@ErikSchierboom
Copy link
Member Author

ErikSchierboom commented Mar 19, 2021

True. Could you reapprove? I can use my superpowers to merge afterwards :)

@ErikSchierboom ErikSchierboom merged commit 3213a80 into exercism:main Mar 19, 2021
@ErikSchierboom ErikSchierboom deleted the add-files-pattern branch March 19, 2021 17:41
@ErikSchierboom
Copy link
Member Author

See #107

@ErikSchierboom ErikSchierboom added the x:size/tiny Tiny amount of work label Oct 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v3-migration 🤖 Preparing for Exercism v3 x:size/tiny Tiny amount of work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants