search

CalPolySEC / wrath-ctf-framework

What? Really? AnoTHer CTF Framework :triangular_flag_on_post:
MIT License
11 stars 4 forks source link

Running PR #8

Closed sig1nt closed 8 years ago

sig1nt commented 8 years ago

This is a running PR from my master, which is in sync with @nickfrostatx, to the repository master. All the code in this has been reviewed already and just needs to be merged.

sig1nt commented 8 years ago

Also, our test cases don't work, so Travis is lying to us

slashnick commented 8 years ago

If the tests are broken, we should probably fix those

sig1nt commented 8 years ago

Do the test's work on your computer? For me they run into problems with the tests not being sandboxed

sig1nt commented 8 years ago

Alright, I have a PR in to @nickfrostatx's branch to fix the tests, then we can revisit a merge

maxzinkus commented 8 years ago

Huh?

On Thu, Sep 1, 2016, 8:59 AM Nick Gonella notifications@github.com wrote:

Closed #8 https://github.com/DeltaHeavy/wrath-ctf-framework/pull/8.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/DeltaHeavy/wrath-ctf-framework/pull/8#event-775379079, or mute the thread https://github.com/notifications/unsubscribe-auth/AFOJX13yVjsWeOmwI5ESLydb2Zz44rY-ks5qlvZfgaJpZM4JyA3J .

Max Zinkus

sig1nt commented 8 years ago

I didn't want to have this PR just sitting there, so we'll reopen it once we get the PR merged that fixes the test cases.