Skip to content

Conversation

ErikSchierboom
Copy link
Member

This makes the practice exercise's structure more in line with the concept exercise's structure

@tasxatzial
Copy link
Member

This seems to be changing the location of the example.clj to ".meta/example.clj"

The PRs that refactor test.clj and scaffold the exercises, still appear to be using "./meta/src/example.clj". Not sure about the PR that verifies the exercises, though.

You aware of all this, right? Are you planning to update this PR once the other PRs are merged?

@ErikSchierboom
Copy link
Member Author

You aware of all this, right? Are you planning to update this PR once the other PRs are merged?

Yes

Copy link
Member

@tasxatzial tasxatzial left a comment

Choose a reason for hiding this comment

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

This appears to fix the formatting in the generator files and it also gets rid of the "src" folder from everywhere.

There's only a minor issue.

The exercises/practice/list-ops/.meta/src/example_lists.clj file, which was my solution to the optional goal for this exercise is gone too :) Can you please create an "extra" folder and put it there?

@ErikSchierboom
Copy link
Member Author

Yeah sure. I'll add it back

@ErikSchierboom
Copy link
Member Author

Done

@ErikSchierboom ErikSchierboom merged commit ca77a6d into main Jan 9, 2025
2 checks passed
@ErikSchierboom ErikSchierboom deleted the simplify-structure branch January 9, 2025 07:36
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.

2 participants