Skip to content

Add stricter configuration defaults#157

Merged
Ninja3047 merged 9 commits into
mainfrom
stricter-config
Jun 14, 2026
Merged

Add stricter configuration defaults#157
Ninja3047 merged 9 commits into
mainfrom
stricter-config

Conversation

@dguido

@dguido dguido commented Jan 26, 2026

Copy link
Copy Markdown
Member

Summary

  • Bump minimum Python version to 3.11 (removes 3.10 from CI matrix)
  • Add security pre-commit hooks: shellcheck, actionlint, zizmor
  • Stricter ruff config: src path for first-party imports, docstring-code-format
  • Stricter coverage: branch coverage enabled, standard exclude patterns
  • Add pip-audit dependency group for vulnerability scanning

Test plan

  • Generated CLI project passes make lint && make test
  • Generated library project passes make lint && make test
  • Pre-commit hooks run successfully (actionlint, zizmor verified)
  • Branch coverage appears in test output
  • uv run pip-audit runs successfully

🤖 Generated with Claude Code

- Bump minimum Python to 3.11, remove 3.10 from CI matrix
- Add security pre-commit hooks: shellcheck, actionlint, zizmor
- Add ruff src path and docstring-code-format settings
- Enable branch coverage with standard exclude patterns
- Add pip-audit dependency group for vulnerability scanning

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Comment thread {{cookiecutter.project_slug}}/.pre-commit-config.yaml Outdated
Ninja3047 and others added 4 commits January 26, 2026 17:03
* main:
  build(deps): bump the actions group across 2 directories with 2 updates (#177)
  build(deps): bump the actions group across 2 directories with 1 update
  build(deps): bump the actions group across 2 directories with 1 update
  build(deps): bump the actions group across 2 directories with 5 updates
  build(deps): bump the actions group across 2 directories with 1 update
  build(deps): bump actions/deploy-pages
  build(deps): bump the actions group across 2 directories with 1 update
  build(deps): bump the actions group across 2 directories with 3 updates
  build(deps): bump actions/upload-artifact
  build(deps): bump actions/download-artifact
  build(deps): bump the actions group across 2 directories with 1 update
  build(deps): bump the actions group across 2 directories with 1 update (#160)
  Change Dependabot schedule from daily to weekly
  build(deps): bump actions/attest-build-provenance
  build(deps): bump the actions group across 2 directories with 1 update
Comment thread .github/dependabot.yml
Comment on lines +21 to +31
- package-ecosystem: pre-commit
cooldown:
default-days: 7
directories:
- "{{cookiecutter.project_slug}}"
groups:
pre-commit:
patterns:
- "*"
schedule:
interval: weekly

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Tested and confirmed works. Generated this PR on my fork ekilmer#2

@ekilmer ekilmer requested a review from Ninja3047 June 13, 2026 22:07
Comment on lines +55 to +58
.PHONY: audit
audit:
uv sync --group audit
uv run pip-audit .

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

uv has a bulit in audit, but this can be a follow up
https://docs.astral.sh/uv/reference/cli/#uv-audit

@Ninja3047 Ninja3047 merged commit 6f0e86a into main Jun 14, 2026
7 checks passed
@Ninja3047 Ninja3047 deleted the stricter-config branch June 14, 2026 18:07
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.

3 participants