Skip to content
Snippets Groups Projects

Refactor

Merged Refactor
refactor into master
Merged Henrik Brosenne requested to merge refactor into master

As all points from the epic refactor issue are closed #47 (closed) I am merging this in to master in order to unblock development.

Merge request reports

Checking pipeline status.

Approval is optional

Merged by Henrik BrosenneHenrik Brosenne 7 years ago (Nov 15, 2017 5:33pm UTC)

Merge details

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
    • Henrik Brosenne
      Henrik Brosenne @hbrosen commented on commit e5d0f2e6
      Author Owner

      After long last, i've found a solution that works and doesn't even depend on pre-commit (screw that). Just add a pre-commit file to .git/hooks with the following content:

      #!/bin/bash

cd backend
unset GIT_DIR
git diff  --cached --name-only --relative | xargs prospector --uses django

      Prospector must be installed, so either the virtualenv must be activated during commit, or it mus be installed system-wide.

      By robinwilliam.hundt on 2017-11-15T10:20:17 (imported from GitLab)

      Edited by Henrik Brosenne
  • Henrik Brosenne merged

    merged

    By Jan Maximilian Michal on 2017-11-15T17:33:53 (imported from GitLab)

  • Henrik Brosenne mentioned in commit fa26fc3e

    mentioned in commit fa26fc3e

    By Jan Maximilian Michal on 2017-11-15T17:33:54 (imported from GitLab)

Please register or sign in to reply