Skip to content

Conversation

@ee7
Copy link
Member

@ee7 ee7 commented Apr 11, 2023

Closes: #2260


Should we remove these, @ErikSchierboom? I think we only had them for historical reasons.

With this PR, there's only one mention of fetch-configlet in this repo:

$ git grep --break --heading 'fetch-configlet'
README.md
252:To download the latest version of the `configlet` tool, please run the [`fetch-configlet`](https://github.com/exercism/configlet/blob/main/scripts/fetch-configlet) bash script or the [`fetch-configlet.ps1`](https://github.com/exercism/configlet/blob/main/scripts/fetch-configlet.ps1) PowerShell script (Windows only). At least one of these scripts should already exist in every track repo's `bin` directory - the script will also download `configlet` to this location. You can then sync the tests by running→

@ee7 ee7 requested a review from ErikSchierboom April 11, 2023 13:13
Copy link
Member

@ErikSchierboom ErikSchierboom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think so

@IsaacG IsaacG merged commit f699acb into exercism:main Apr 12, 2023
@ee7 ee7 deleted the fetch-configlet-remove branch April 12, 2023 15:17
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.

fetch-configlet: consider removing scripts from this repo

5 participants