Scripta-Qumranica-Electronica / Scrollery-website

SQE website
MIT License
3 stars 2 forks source link

Cgi patch #76

Closed Bronson-Brown-deVost closed 6 years ago

Bronson-Brown-deVost commented 6 years ago

Using $::{'stringWithNameOfSubroutine'} does not work on the SQE VM server. I have now reverted back to a hash table for routing requests. I also include small updates to the webhook which fix some errors in the build process related to updates of SQE_DB_API.

Bronson-Brown-deVost commented 6 years ago

The travis-ci tests failed when trying to report to coveralls with the message "Bad response: 422 {"message":"Couldn't find a repository matching this job.","error":true}". @sjones6 can you take a look at our Coveralls setup? Something seems to be broken there.

sjones6 commented 6 years ago

Hmm. That's a new one. I requeued the build but it looks like everything is failing for the same reason. It's something about authentication but I'm not sure what (Travis, Github, Coveralls ...).

sjones6 commented 6 years ago

ok, refreshed the Coveralls token and will see if that helps.

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 87.056% when pulling cb51380e24ec42c8915c8ac5b4e031d2ef84d78a on cgi-patch into e8dbd037c3d919c44f5f061969ac33aede97d163 on master.

sjones6 commented 6 years ago

Looks like refreshing the token did the trick. Coveralls is still somewhat slow in reporting (takes a few mins) but looks like we're back in business.