016/solution #10

Closed
wants to merge 13 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@bengitscode
Contributor

bengitscode commented Mar 13, 2017

Refresh solution for cohort 016

Ben Adamski added some commits Mar 9, 2017

Ben Adamski
Update from challenge-template
-  Refresh for cohort 016.
Ben Adamski
Update from node-template
-  Refresh for cohort 016.
-  Update complete gitignore file with:
 -  macosx.
 -  node.
 -  ruby.
 -  python.
Ben Adamski
Add necessary chai and expect imports to tests
-  Was this removed in update?
Ben Adamski
Change all instance of .toEqual to .to.equal
-  This syntax has been updated to` to.equal`
-  Tests will not pass with old syntax.
Ben Adamski
Correct typo in README whether or not
-  Changed whether of not.
  -  to: whether or not.
-  Death to typos!

@bernardlee bernardlee requested a review from bengitscode Mar 13, 2017

Ben Adamski added some commits Mar 14, 2017

Ben Adamski
Silence rubocop in spec/challenge_spec.rb
-  Updates with preferred single quotes and array synatax.
Ben Adamski
Remove extraneous files
-  Address review in #9
Ben Adamski
Correct javascript test
-  Address review in #9
-  Missed updated syntax correction.
Ben Adamski
Add frozen_literal:true comments
-  lib/challenge.rb.
-  spec/challenge_spec.rb.
-  Address #9.
Ben Adamski
Correct javascript challenge test
-  Missed toBeNull to to.be.null change.
Ben Adamski
Update solution
-  Refresh for cohort 016.

@bengitscode bengitscode force-pushed the 016/solution branch to 0575f06 Mar 14, 2017

@bernardlee bernardlee closed this Mar 14, 2017

@bernardlee bernardlee deleted the 016/solution branch Mar 14, 2017

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