Skip to content

fix: Small fixes from the October 2026 review (0.8.9) - #76

Merged
csgillespie merged 2 commits into
mainfrom
fix/72-small-fixes
Oct 16, 2026
Merged

csgillespie merged 2 commits into
mainfrom
fix/72-small-fixes

Conversation

@csgillespie

Copy link
Copy Markdown
Contributor

0.8.8 → 0.8.9

  • Any exported class whose name starts with check_ is run as a check, as the README says. Before, only check_deploy* classes were.
  • check() prints the user summary once.
  • The Git-backed check deploys with its own title, so its content is deleted afterwards.
  • Python clean-up deletes only content owned by the auditor.
  • The report's old-users table lists users who have never logged in. Users inactive for exactly 6 or 12 months are no longer left out.
  • The README's renv advice uses a temporary cache, instead of deleting the user's whole renv cache.

Also: R6 and utils in Imports, no self-referencing debug_level defaults, and corrected Flask/Shiny descriptions and plumber class name. read.dcf() is base R, so it needed nothing.

Closes #72

🤖 Generated with Claude Code

https://claude.ai/code/session_01BEZ4ZUv51gFQjAe1WePbJF

csgillespie and others added 2 commits October 16, 2026 17:14
- Find checks with audit.base::init_r6_checks(), so any exported check_ class runs, as the README says
- Import R6 and utils; call audit_details() once
- deploy_git() uses its title, so cleanup finds the content
- Drop the self-referencing debug_level defaults
- cleanup_python() only deletes the auditor's own content
- The old-users table lists users who have never logged in, and 6 and 12 months exactly
- Fix the Flask and Shiny descriptions and the plumber class name
- README: use a temporary renv cache instead of deleting the user's

Closes #72

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01BEZ4ZUv51gFQjAe1WePbJF
@csgillespie csgillespie self-assigned this Oct 16, 2026
@csgillespie
csgillespie merged commit f219f11 into main Oct 16, 2026
5 checks passed
@csgillespie
csgillespie deleted the fix/72-small-fixes branch October 16, 2026 16:21
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.

Small fixes from the October 2026 review

1 participant