016/solution #12

Closed
wants to merge 6 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@bengitscode
Contributor

bengitscode commented Mar 11, 2017

Update solution from origin solution

Refresh for cohort 016

Ben Adamski added some commits Mar 10, 2017

Ben Adamski
Update from challenge-template
-  Refresh for cohort 016.
-  Removes #Ruby section from gitignore
  -  Will be patched back in on ruby-template update.
Ben Adamski
Update from ruby-template
-  Refresh for cohort 016.
 -  Adds ruby section back into gitignore.
Ben Adamski
Move morse_code data into separate file
-  Move data into separate file and require in challenge.rb
-  Close #8
Ben Adamski
Appease rubocop
-  Address lingering frozen_string_literal comment in
challenge_spec.rb.
Ben Adamski
Update solution from origin/solution
-  Refresh for cohort 016
-  Passing all tests.
Ben Adamski
Appropriate frozen_string_literal comments
-  False in challenge.rb
-  True in challenge .rb
-  Must be false to modify original string.

@bernardlee bernardlee requested a review from bengitscode Mar 13, 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