Skip to content

Remove all tkinter GUI components - #1

Merged
gdevenyi merged 1 commit into
legacyfrom
remove-gui-components
Mar 30, 2026
Merged

Remove all tkinter GUI components#1
gdevenyi merged 1 commit into
legacyfrom
remove-gui-components

Conversation

@gdevenyi

@gdevenyi gdevenyi commented Mar 30, 2026

Copy link
Copy Markdown
Member
  • Delete brkraw/ui/ (active GUI, 7 files)
  • Delete brkraw/app/viewer/ (dead duplicate, 7 files)
  • Delete imgs/brkraw_gui.png screenshot asset
  • Remove gui subcommand from CLI (brkraw/scripts/brkraw.py)
  • Remove viewer optional dependency from pyproject.toml
  • Update all = [brkraw[tonii]] (removed brkraw[viewer])
  • Fix xnippet dependency: git URL -> PyPI (xnippet>=0.1.1)
  • Update CLAUDE.md and README.md to remove GUI references

Summary

Explain what this PR changes and why.

Type of change

  • Bug fix
  • New feature
  • Refactor/maintenance
  • Docs update
  • CI/CD/workflow

Checklist

  • I ran tests locally (pytest) and they pass
  • I updated/add tests that cover this change
  • I updated documentation where relevant (README, docstrings)
  • I considered backward compatibility and breaking changes
  • I followed existing code style and conventions

How to test

Provide steps or commands for reviewers to verify the change.

pytest -q

- Delete brkraw/ui/ (active GUI, 7 files)
- Delete brkraw/app/viewer/ (dead duplicate, 7 files)
- Delete imgs/brkraw_gui.png screenshot asset
- Remove gui subcommand from CLI (brkraw/scripts/brkraw.py)
- Remove viewer optional dependency from pyproject.toml
- Update all = [brkraw[tonii]] (removed brkraw[viewer])
- Fix xnippet dependency: git URL -> PyPI (xnippet>=0.1.1)
- Update CLAUDE.md and README.md to remove GUI references
@gdevenyi
gdevenyi merged commit 7b3d3cc into legacy Mar 30, 2026
1 of 21 checks passed
@gdevenyi
gdevenyi deleted the remove-gui-components branch March 30, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant