Skip to content

use uv and ruff for dev tooling + add vscode settings and recommended extensions#184

Merged
swansonk14 merged 19 commits into
swansonk14:mainfrom
utkashd:utkashd/astralize
Jun 21, 2026
Merged

use uv and ruff for dev tooling + add vscode settings and recommended extensions#184
swansonk14 merged 19 commits into
swansonk14:mainfrom
utkashd:utkashd/astralize

Conversation

@utkashd

@utkashd utkashd commented May 27, 2026

Copy link
Copy Markdown
Contributor

Hello, I noticed this repo's tooling is a bit dated, so I created this PR to modernize the dev tooling.

  • switch to using uv for dependency management
  • replace black + flake8 with ruff for linting and formatting
  • add VS Code workspace settings, extensions

Callouts:

  • in pyproject.toml I set exclude-newer = "7 days" to lessen supply chain risk
  • it looks like this repository intentionally doesn't have a .vscode/. That seems odd, but maybe there's a good reason. If so, I'll remove the vscode changes
  • all changes to code should only be formatting changes. The meat of this PR is in pyproject.toml, .vscode/, and the README

How to validate:

uv run ruff check   # "All checks pass!"
uv run ruff format  # "14 files left unchanged"
uv run pytest       # all tests should pass

@utkashd utkashd changed the title uv + ruff + vscode settings + extensions wip: uv + ruff + vscode settings + extensions May 27, 2026
@utkashd utkashd marked this pull request as draft May 27, 2026 10:13
@utkashd utkashd changed the title wip: uv + ruff + vscode settings + extensions use uv and ruff for dev tooling + add vscode settings and recommended extensions May 28, 2026
Comment thread src/tap/tap.py
from shlex import quote, split
from types import MethodType, UnionType
from typing import (
Annotated,

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

unused import; I assume this is safe to remove

@utkashd utkashd marked this pull request as ready for review May 28, 2026 17:36
Comment thread README.md Outdated
Comment thread README.md Outdated
@swansonk14

Copy link
Copy Markdown
Owner

This looks great, thank you so much @utkashd!

--JK

@swansonk14 swansonk14 merged commit 2c39de2 into swansonk14:main Jun 21, 2026
15 checks passed
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.

2 participants