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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃 Add stub files to all exercises #968

Closed
wants to merge 1 commit into from

Conversation

ErikSchierboom
Copy link
Member

In this PR, we created (empty) stub files for all exercises that didn't yet have them.

The lack of stub file generates an unnecessary pain point within Exercism, contributing a significant proportion of support requests, making things more complex for our students, and hindering our ability to automatically run test-suites and provide automated analysis of solutions.

The original discussion for this is at exercism/discussions#238.

We will automatically merge this PR in two week's time if it has not been merged by track maintainers at that point 馃檪

Tracking

exercism/v3-launch#33

@ErikSchierboom ErikSchierboom added the v3-migration 馃 Preparing for Exercism v3 label May 4, 2021
@petertseng
Copy link
Member

All exercises in this track already have stub files.

If they're not named what they should be, that's an interesting complaint, but the fix to that should be to rename the existing one, not create new ones. Creating a new empty one will confuse the student further because then there will be two files in the src directory and the student won't know which one is supposed to be edited.

Here is why all exercises in this track already have stub files:

The number of exercises is:

$ ls -1d exercises/practice/* | wc -l
101

The number of .hs files in a src directory is:

$ ls exercises/practice/*/src/*.hs | wc -l
101

But we need to make sure those files map 1-to-1 with exercises, rather than some exercises having multiple and some having zero. So extract the exercise name of each.

$ ls exercises/practice/*/src/*.hs | cut -d/ -f3 | sort | uniq | wc -l
101

Okay, so each existing exercise does indeed have a stub file.

@petertseng petertseng closed this May 4, 2021
@ErikSchierboom
Copy link
Member Author

@petertseng No idea what went wrong there. Sorry!

@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.

None yet

2 participants