Skip to content

Remove banned Python git-cola integration (no SaltStack exemption remains) #31

@hyperpolymath

Description

@hyperpolymath

Problem

bitfuckit/integrations/git-cola/actions.py (105 lines) is a Python file. Python is banned by the language policy; the only historical carve-out was SaltStack, and SaltStack is gone (infra moved to Terraform + Ansible). So there is no valid exemption and this file deterministically fails the antipattern-check CI ("Check for Python (non-SaltStack)" step) on every branch.

This is a pre-existing failure (also failed identically on PRs #29 and #30); it is not caused by the V-removal work in #30.

What the file does

It's a git-cola plugin (from cola import cmds) that registers GUI actions which just shell out to the bitfuckit CLI (bitfuckit pr create, bitfuckit pr list, etc.). No real logic — pure thin wrappers.

Recommended resolution

git-cola's plugin API is Python-only, so this cannot be a like-for-like Python→Rust/Julia port. But it doesn't need to be a plugin: git-cola supports user-defined guitool / shell-command actions (git config guitool.<name>.cmd, or ~/.config/git-cola/actions) with zero Python. Since the plugin only invokes the bitfuckit CLI, replace it with:

  • a small POSIX shell snippet / git-cola guitool config that calls bitfuckit, and
  • update the install docs accordingly,
  • then delete actions.py.

Acceptance criteria

  • bitfuckit/integrations/git-cola/actions.py removed
  • Equivalent git-cola actions provided via shell/guitool config (no Python)
  • Install docs updated
  • antipattern-check Python step passes with no exemption needed

See also the companion CI/policy cleanup issue (drop the obsolete SaltStack Python carve-out).

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions