Skip to content

presenting clean package.yaml to students #420

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

Closed
23 tasks
petertseng opened this issue Nov 7, 2016 · 5 comments
Closed
23 tasks

presenting clean package.yaml to students #420

petertseng opened this issue Nov 7, 2016 · 5 comments

Comments

@petertseng
Copy link
Member

petertseng commented Nov 7, 2016

Since #396 we can move our examples to a separate directory with its own package.yaml to prevent showing students extra elements in dependencies in package.yaml

Here are the current exercises with non-empty dependencies.

It should be pretty easy to script the process of moving them all at once. The question before doing that: Are there any exercises where we should just keep the package.yaml as-is?

  • atbash-cipher: split
  • bank-account: stm
  • connect: array
  • crypto-square: split
  • food-chain: array
  • go-counting: array
  • grade-school: containers
  • kindergarten-garden: containers, split
  • largest-series-product: safe
  • lens-person: lens
  • linked-list: stm
  • ocr-numbers: containers, split
  • parallel-letter-frequency: parallel
  • rna-transcription: containers
  • robot-name: random
  • saddle-points: containers
  • say: array
  • scrabble-score: containers
  • sgf-parsing: attoparsec
  • sieve: vector
  • simple-cipher: random
  • triangle: containers
  • wordy: attoparsec, text
@rbasso
Copy link
Contributor

rbasso commented Nov 7, 2016

IMHO, the only reason to provide a package.yaml with dependencies in the library section is if is impossible to solve without it, because of the test suite demands it, or if the stub solution uses it:

  • go-counting needs containers.
  • kindergarten-garden needs containers.
  • ...

I haven't checked all of them yet.

@rbasso
Copy link
Contributor

rbasso commented Nov 7, 2016

I removed all dependencies that where not used by the stub solutions or demanded by the test suites, but the process was far from automatic. 😫

It should be pretty easy to script the process of moving them all at once.

@petertseng
Copy link
Member Author

but the process was far from automatic.

Oops :<

Oh well, at least we only have to do it this once, right? So this issue is closed now, right?

@petertseng
Copy link
Member Author

Oh wait, you were doing that in #423. Okay, I'll leave it open until then.

@petertseng
Copy link
Member Author

Coming in real quick to mark down a reference - this fixes a flaw first noted in #189 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants