Separate script and library #7

Closed
laurenfazah opened this Issue Aug 29, 2016 · 0 comments

Comments

Projects
None yet
3 participants
@laurenfazah

laurenfazah commented Aug 29, 2016

During discussion, similar to how it's done in the diagnostic

@jrhorn424

This comment has been minimized.

Contributor

jrhorn424 commented Oct 7, 2016

Does this mean separate files into a data directory? Or something about lib and scripts? or bin? 🤔

@gaand

This comment has been minimized.

gaand commented Oct 7, 2016

I think add a bin for the things that run from the command line, lib contains code required and invoked by those.

Better separation of concerns, etc.

@jrhorn424 Please let me know if you need more clarification.

raq929 pushed a commit that referenced this issue Feb 24, 2017

raq929 pushed a commit that referenced this issue Feb 24, 2017

racheltstevens
Add solution branch.
With separated concerns per #7
Refresh for cohort 016.

raq929 pushed a commit to ga-wdi-boston/node-api-promises that referenced this issue Feb 27, 2017

racheltstevens
Separate concerns
Per ga-wdi-boston/node-api#7
Fixes randomizer script ga-wdi-boston/node-api#18
Moves process.argv issues to `bin/<file-name>`
Keeps other logic in `lib/<file-name>`
Uses string interpolation in `lib/hey-yall.js`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment