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

Next step with analyzer: split into libraries #2

Open
glennj opened this issue Sep 27, 2021 · 2 comments
Open

Next step with analyzer: split into libraries #2

glennj opened this issue Sep 27, 2021 · 2 comments

Comments

@glennj
Copy link
Contributor

glennj commented Sep 27, 2021

The shellcheck analysis should be 1 of a number of steps in the analysis.

Next up:

  • move the shellcheck analysis out of bin/run.sh into a lib/ directory,
  • add an exercise-specific analysis
    • two-fer is a good starter, check if submission uses $(var:-you} in some form
  • determine what language will be appropriate for handling the various data structures: continue with ruby?
  • is this the time to add a representer?
@IsaacG
Copy link
Member

IsaacG commented Sep 27, 2021

two-fer is a good starter, check if submission uses $(var:-you} in some form

That presupposes that this exercise is about learning parameter expansions vs conditional and breaking up the logic into reasonable blocks. I find (( $# )) && who=$1 || who=you; echo "One for $who, one for me." is a pretty solid solution, too, and teaches students a fair bit, too

@ErikSchierboom
Copy link
Member

is this the time to add a representer?

I'd argue that the representer might be more important than the analyzer, so starting with the representer seems like a good idea! That said, we still have some work to do on the website to fully integrate representers, but it might be nice to start with the representer anyways.

That presupposes that this exercise is about learning parameter expansions vs conditional and breaking up the logic into reasonable blocks. I find (( $# )) && who=$1 || who=you; echo "One for $who, one for me." is a pretty solid solution, too, and teaches students a fair bit, too

These are exactly the types of discussions to engage in once there is a basic analyzer 👍

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

No branches or pull requests

3 participants