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

🤖 Sync exercise files keys based on file path patterns #25

Merged

Conversation

ErikSchierboom
Copy link
Member

To help tracks define the files in the exercise's .meta/config.json file, we've added support for defining these file patterns in the track's config.json file. See this PR.

This PR updates the file paths defined in the files property of each exercise's .meta/config.json file according to the file patterns defined in the track's config.json file.

We only update a file path in the .meta/config.json file if:

  • The file path pattern is a non-empty array in the config.json file
  • The file path pattern is either not set or an empty array in the .meta/config.json file

This means that exercises that had already defined files in the .meta/config.json won't be touched in this PR.

Note that this PR is just an easy way for tracks to populate the files in the .meta/config.json files. Tracks are completely free to add/change/remove the files in their .meta/config.json files.

In the future, we'll update configlet to include this functionality. If you'd like me to re-run the script because changes have been made, feel free to ping me on Slack (@ErikSchierboom).

Tracking

exercism/v3-launch#20

@ErikSchierboom ErikSchierboom added the v3-migration 🤖 Preparing for Exercism v3 label Apr 7, 2021
@iHiD iHiD merged commit cc47c17 into exercism:main Apr 7, 2021
@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