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.

Sign up for GitHub

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

Jump to bottom

Self Assessment #3

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Self Assessment #3

wants to merge 30 commits into from

Conversation

john-cooper-2
Copy link

Completed Recursion prompts

harveysanders and others added 30 commits June 28, 2018 15:08
@harveysanders
(chore) Pend non required tests
5bbd68c
@jaykindell
adds gitignore & package.json
3f131bd
@jaykindell
reporter - json
d5c8c05
@jaykindell
adds tests in /tests
9addd59
@jaykindell
adds exports for testing
a33c772
@jaykindell
just the package-lock
d8cecf8 
 Please enter the commit message for your changes. Lines starting
@jaykindell
requires exercise modules in tests
043dfd4
Merge pull request #24 from OperationSpark/feat/nodeTests
a2492de 
Feat/node tests
@jaykindell
adds karma config & global location variable
b04e257
@jaykindell
remove folder with node tests
3401210
@harveysanders
Merge pull request #31 from OperationSpark/feat/karmaTest
11e229c 
Feat/karma test
@cainwatson
(chore) Add mocha test script
9060734
@cainwatson
(chore) Use test:travis cmd in travis yml
27a3fa5
@cainwatson
(chore) Add test files
99562bc
@cainwatson
(fix) Use proxies for node recursion callCount
4101afb
@cainwatson
(cleanup) Change createProxy to createSpyProxy + update wording
fc0f7a9
@cainwatson
(fix) Use plain mocha for travis intead of karma
a56687b
@cainwatson
Remove pending tests
4d39579
@cainwatson
Add where students work in README
2d5eb31
(chore) Remve old os tool setup
28b163c
(chore) Use yarn test in for travis
32fc0f9
Merge pull request #32 from OperationSpark/os-grading
21410ac 
OS Tool grading
fix: Not destructuring for c9 node version
260265d
Fix #35 to run test instead of showing as pending
1e58052 
Fix #35 to run test instead of showing as pending
update to run prompt 35
e0c4dc0 
update to run prompt 35 as it says on the directions.
remove extra declaration of numToText
ed13b14
fix test file
73f66c2
fix const assignment for numToText
bda8877
fix proxy for numToText
628487e
@RyGuy-Coder
Update recursion.js
7abc2e1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@john-cooper-2 @harveysanders @jaykindell @cainwatson @RyGuy-Coder