Skip to content
GitLab
  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • Tor Tor
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 320
    • Issues 320
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 37
    • Merge requests 37
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • The Tor Project
  • Core
  • TorTor
  • Issues
  • #40019
Closed
Open
Created Jul 01, 2020 by David Goulet@dgoulet🔆Owner

git hooks: Stop running practracker constantly

Practracker runs on every commit with the pre-commit hook. It does not run on maint- branches but it runs on master but also on EVERY development branch anyone does.

The reason is simple, it is that we never merge things that breaks practracker into master so makes sense that every dev branch is tested.

However, I'm arguing here that running practracker constantly is a waste of time and extremely annoying for development.

There is value in practracker but I think we could simply run it once in a while, maybe every release, update the exception file and be done with it. There is even an argument to say that one should only run it in order to measure our technical debt in some ways or wanting to start fixing part of it.

Again, I don't see value in constantly having a perfectly up to date exceptions.txt file. There is value that we have the tool available in our repository but not being run all the time. Yes, it can break but this is why we can simply run it as part of our release process.

Thus I vote to remove it from our pre-commit-git.hook file.

Thoughts @tpo/core ?

To upload designs, you'll need to enable LFS and have an admin enable hashed storage. More information
Assignee
Assign to
Time tracking