Remove "Commit" code from server

[aeksco]

The "Commit" functionality is barely functional for practical purposes and adds very little value to Observatory from an administrative perspective. We should remove the following files:

• Remove server/api/commit/*

• Remove server/workers/github.py (remove workers directory as well)

• Remove app.use('/api/commits', require('./api/commit')); from server/routes.js

• Remove require('../server/api/commit/commit.spec'); from test/index.js

• Remove commits method from server/api/user/user.controller.js

• Remove commits route from server/api/user/index.js

• Remove stats method from server/api/user/user.controller.js

• Remove the following from /server/api/attendance/attendance.spec.js:

  describe('GET /api/commits', function() {
  
  it('should respond with JSON array', function(done) {
  request(app)
    .get('/api/commits')
    .expect(200)
    .expect('Content-Type', /json/)
    .end(function(err, res) {
      if (err) return done(err);
      res.body.should.be.instanceof(Array);
      done();
    });
  });
  });

Track each removal in a separate commit (that way you get lots of commits :+1:) and run npm test after each change to ensure things are passing spec (ignore failing user tests for first_name and last_name, that was my bad)

[aeksco]

Resolved