Skip to content

Fix stale config: file paths, py-version target, duplicate checkout, classifier#525

Open
hmacdope wants to merge 4 commits intomainfrom
fix/quick-wins
Open

Fix stale config: file paths, py-version target, duplicate checkout, classifier#525
hmacdope wants to merge 4 commits intomainfrom
fix/quick-wins

Conversation

@hmacdope
Copy link
Copy Markdown
Contributor

Summary

A collection of stale/incorrect config fixes caught during a codebase audit.

  • .lgtm.yml: remove versioneer.py entry (project uses versioningit); fix generated-file path openadmet_models/_version.pyopenadmet/models/_version.py
  • .pre-commit-config.yaml: fix files: patterns for black, isort, and flake8 from ^openadmet_models^openadmet (the hooks were silently not running on any source files); bump pyupgrade from --py39-plus--py310-plus to match requires-python = ">=3.10"
  • GPU_CI_integration.yaml: remove duplicate actions/checkout@v4 step
  • pyproject.toml: bump Development Status classifier from 1 - Planning3 - Alpha

Test plan

  • Pre-commit hooks now trigger on openadmet/ source files
  • CI passes

🤖 Generated with Claude Code

hmacdope and others added 2 commits April 14, 2026 11:48
…ifier

- .lgtm.yml: remove versioneer.py reference (project uses versioningit);
  fix generated path openadmet_models/ -> openadmet/models/
- .pre-commit-config.yaml: fix black/isort/flake8 file patterns from
  ^openadmet_models to ^openadmet; bump pyupgrade target --py39-plus ->
  --py310-plus to match requires-python = ">=3.10"
- GPU_CI_integration.yaml: remove duplicate actions/checkout@v4 step
- pyproject.toml: bump classifier from "1 - Planning" to "3 - Alpha"

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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